-
Notifications
You must be signed in to change notification settings - Fork 44
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
18951 Change amalgamation to amalgamationApplication #750
18951 Change amalgamation to amalgamationApplication #750
Conversation
Kevin, you need to import the latest shared enums. That's why the build-check has failed. Next time, please run the build locally (and also the unit tests and linting) before assigning reviewers to the PR. |
@eve-git @ozamani9gh We would like to merge this PR as soon as possible. Are you able to review this PR? Can we merge it ourselves? |
/gcbrun |
Temporary Url for review: https://namerequest-dev--pr-750-7lf1pe42.web.app |
Issue #: /bcgov/entity#18951
Description of changes:
Amalgamation Object will be like this
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namerequest license (Apache 2.0).