Skip to content

Conversation

@VenissaCarolQuadros
Copy link
Member

@VenissaCarolQuadros VenissaCarolQuadros commented Jun 16, 2025

This PR includes the following changes:

  1. Merge changes from SVGEdit repo. This includes dependency updates and code changes.
  2. Include rollup steps in Dockerfile. These steps need to be removed from the IMAGE TAT service Dockerfile..
    This step makes the build process occur within this repo which is ideal as there were cases in the past where changes in dependencies would install but not properly build on docker (but the build only occurred in the IMAGE-server workflow making them harder to test/ identify). With these changes, such failures can be caught and rectified at the first build stage.

These changes were tested by using an override on unicorn along with the changes made in this PR updating the IMAGE TAT service Dockerfile and ensuring that the build works and pushes data correctly to the Monarch with this photo and a hand drawn example.

@jaydeepsingh25 jaydeepsingh25 merged commit eac9373 into main Jun 18, 2025
2 checks passed
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.

6 participants