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

[CHORE] - update VerificationMethod and .env-example #3272

Merged
merged 23 commits into from
Oct 21, 2024

Conversation

Jsyro
Copy link
Collaborator

@Jsyro Jsyro commented Oct 18, 2024

Env changes have been made in openshift, should be made locally for DEV to match.

The verificationmethod had the wrong fragment in it, it was hard coded for how indy ledgers create diddocuments, but the tdw server does it differently, now take verification method and derive did from that.

@Jsyro Jsyro marked this pull request as ready for review October 18, 2024 16:44
@Jsyro Jsyro added the 👍 Ready for review Pull request has been double checked by the author and is ready for comments and feedback. label Oct 18, 2024
@Jsyro Jsyro changed the title [CHORE] - update .env-example to new values [CHORE] - update VerificationMethod and .env-example Oct 18, 2024
stop using sovrin did, just use did:web from tdw server.
Comment on lines +295 to +296
"CHIEF_PERMITTING_OFFICER_DID_WEB_VERIFICATION_METHOD",
"CHIEF_PERMITTING_OFFICER_DID_WEB_VERIFICATION_METHOD")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why two lines here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2nd params is the default if not provided.

App won't start if the second param is not provided.

slightly more developer friendly this way, .env-example has the appropriate values.

Copy link

sonarcloud bot commented Oct 18, 2024

Quality Gate Failed Quality Gate failed for 'bcgov-sonarcloud_mds_core-api'

Failed conditions
66.7% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

Copy link
Collaborator

@matbusby-fw matbusby-fw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🫎

@Jsyro Jsyro merged commit 6875c65 into develop Oct 21, 2024
11 of 12 checks passed
@Jsyro Jsyro deleted the feature/untp-dcc-cleanup branch October 21, 2024 19:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👍 Ready for review Pull request has been double checked by the author and is ready for comments and feedback.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants