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

OCL initiate parse zip to json #1581

Draft
wants to merge 17 commits into
base: develop
Choose a base branch
from

Conversation

vsvishalsharma
Copy link
Contributor

@vsvishalsharma vsvishalsharma commented Feb 19, 2025

Pull Requests Requirements

  • The PR title includes a brief description of the work done, including the
    Issue number if applicable.
  • The PR includes a video showing the changes for the work done.
  • The PR title follows conventional commit label standards.
  • The changes confirm to the OpenElis Global x3 Styleguide and design
    documentation.
  • The changes include tests or are validated by existing tests.
  • I have read and agree to the Contributing Guidelines of this project.

Summary

To see the json output put the zip file location in the org.openelisglobal.ocl.zip.path inside the file src/main/resources/application.properties

then need to

run mvn test -Dtest=OclZipImporterIntegrationTest

update the log4j2.properties file change

logger.rolling.level from error to info
rootLogger.level from error to info

Screenshots

Screenshot from 2025-02-19 16-34-51

Related Issue

[Add a link to the related issue or mention it here if applicable]

Other

[Add any additional information or notes here]

@vsvishalsharma vsvishalsharma marked this pull request as draft February 19, 2025 12:17
@vsvishalsharma vsvishalsharma changed the title Ocl initiate Ocl initiate parse zip to json Feb 19, 2025
@vsvishalsharma vsvishalsharma changed the title Ocl initiate parse zip to json OCL initiate parse zip to json Feb 19, 2025
@vsvishalsharma
Copy link
Contributor Author

To test the parsing we can download a test zip file from here

@reagan-meant reagan-meant requested a review from mozzy11 February 25, 2025 12:33
@github-actions github-actions bot added the merge conflict Merge Conflicts label Feb 26, 2025
Copy link

👋 Hi, @vsvishalsharma,
Conflicts have been detected against the base branch. Please rebase your branch against the base branch.


This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

@github-actions github-actions bot removed the merge conflict Merge Conflicts label Feb 26, 2025
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.

1 participant