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

Revert to latest stable releases of ozone and OpenMRS docker image #57

Merged
merged 6 commits into from
Nov 4, 2024

Conversation

jnsereko
Copy link
Contributor

@jnsereko jnsereko commented Oct 29, 2024

Requirements

  • This PR has a title that briefly describes the work done including the ticket/issue number. If there is a ticket, make sure your PR title includes a conventional commit label. See existing PR titles for inspiration.
  • My work conforms to the OpenMRS 3.0 Styleguide and design documentation.
  • My work includes tests or is validated by existing tests. If there is no automated tests, I tested this PR on my localhost.
  • I included a screenshot or video of the change. This will help a lot the reviewers to test your changes and accelerate the approval.

Summary

Requirement is use:

Screenshots or video

Screenshot 2024-10-29 at 09 39 50 Screenshot 2024-10-29 at 10 01 45 Screenshot 2024-10-29 at 10 02 42

Related Issue

#56

Other

@jnsereko
Copy link
Contributor Author

@michaelbontyes what do you think about this??
@WodPachua @suubi-joshua, could you kindly test if all is ok with the frontend?

Found errors:

  • I can't save a form

@WodPachua
Copy link
Contributor

WodPachua commented Oct 29, 2024

@jnsereko on my side, the frontend starts just fine... I tried on gitpod

@jnsereko
Copy link
Contributor Author

  • Noticed the issues with the forms

Which one exactly?

  • Not sure why your frontend assembly doesn't have billing?

Should you kindly share your spa assemble file before a this PR?

@WodPachua
Copy link
Contributor

WodPachua commented Oct 29, 2024

Which one exactly?

I see only two forms here... the first cant save... the second shows the errors as in the screenshot
image
image

@WodPachua
Copy link
Contributor

Should you kindly share your spa assemble file before a this PR?

I think the only two apps missing are the billing and stock, which we can just add in the other assembly file you edited

@WodPachua WodPachua self-requested a review October 29, 2024 14:33
@michaelbontyes michaelbontyes self-requested a review October 30, 2024 02:15
@michaelbontyes
Copy link
Member

@WodPachua , @jnsereko , are you able to run with HTTPS on your local? It keeps failing on my side. Thanks

@jnsereko
Copy link
Contributor Author

jnsereko commented Oct 30, 2024

Hey @WodPachua, how about now? I am looking into the forms
Screenshot 2024-10-30 at 10 52 01

@jnsereko
Copy link
Contributor Author

@WodPachua , @jnsereko , are you able to run with HTTPS on your local? It keeps failing on my side. Thanks

@suubi-joshua, should i raise a PR for this??

@suubi-joshua
Copy link
Contributor

suubi-joshua commented Oct 30, 2024

@WodPachua , @jnsereko , are you able to run with HTTPS on your local? It keeps failing on my side. Thanks

@suubi-joshua, should i raise a PR for this??

@jnsereko Can we have a call this evening to discuss this and why its failing.

@WodPachua
Copy link
Contributor

Hey @WodPachua, how about now? I am looking into the forms Screenshot 2024-10-30 at 10 52 01

This is now fine...

@WodPachua
Copy link
Contributor

WodPachua commented Oct 30, 2024

@WodPachua , @jnsereko , are you able to run with HTTPS on your local? It keeps failing on my side. Thanks

Am not sure how am to check this on gitpod... since my local PC environment just riots 🥲

  • However I noticed that the frontend now has issues, it kinda fails after a while... not sure why, the logs don't seem to say anything @jnsereko when I test these latest updates made.
  • Also noticed this backend dependency error which was initially resolved last time I tested, is back again... something about the fhir version in datavisualizer not matching the rest I think... not really sure...
    image
    image

@tendomart
Copy link
Contributor

tendomart commented Nov 4, 2024

@jnsereko got some conflicts here

@michaelbontyes
Copy link
Member

michaelbontyes commented Nov 4, 2024

@WodPachua @jnsereko, can we solve this one for the team in Burundi to see it this week? We can temporary disable the SSL if it helps. Thanks

@michaelbontyes michaelbontyes merged commit a0353ec into MadiroGlobalHealth:main Nov 4, 2024
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.

5 participants