Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: 3512 load files from jar #4393

Merged
merged 8 commits into from
Oct 24, 2024
Merged

Conversation

connoratrug
Copy link
Contributor

What are the main changes you did:

Closes #3512

  • fix loading files from within a jar ( scan files in '_files' folder with jar supporting libs )
  • add example files to catalogue profile demodata

how to test:

  • explain here what to do to test this (or point to unit tests)

todo:

  • updated docs in case of new feature
  • added/updated tests
  • added/updated testplan to include a test for this fix, including ref to bug using # notation

getBinaryFileWrapper impl did not work within jar file ( ci and prod runs as jar)
by skipping to getResourceAsStream the file van be read from the jar, but folder can not be listed this way

either pas file name ( extentions and type ) to getBinaryFileWrapper or find a way to scan the files with in the jar
- keep in backend for tesing
Copy link

@connoratrug connoratrug requested review from mswertz and harmbrugge and removed request for mswertz October 24, 2024 12:52
@connoratrug connoratrug merged commit 6b270fb into master Oct 24, 2024
6 checks passed
@connoratrug connoratrug deleted the fix/3512-load-files-from-jar branch October 24, 2024 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix(demo): As catalogue data manager I would like to include files in the test/demo data on the repository
2 participants