-
Notifications
You must be signed in to change notification settings - Fork 142
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
Add and remove OpenID credentials from anchor #2810
Merged
sea-snake
merged 69 commits into
main
from
sea-snake/openid-google-add-remove-from-anchor
Jan 28, 2025
Merged
Changes from 60 commits
Commits
Show all changes
69 commits
Select commit
Hold shift + click to select a range
464c0d2
Add OpenID credential storage in stable structures and anchor managem…
sea-snake 7db91b5
- OpenID credentials in stable structures.
sea-snake 7e749e7
🤖 cargo-fmt auto-update
github-actions[bot] 1a92adb
🤖 npm run generate auto-update
github-actions[bot] cca4497
Fix error in archive operation compat
sea-snake eaf5646
Fix error in anchor tests
sea-snake db90889
Add OpenID credentials to did.
sea-snake b3534cb
🤖 npm run generate auto-update
github-actions[bot] 98e3455
Add OpenID credentials operations to archive did.
sea-snake 279a73a
Merge remote-tracking branch 'origin/sea-snake/openid-google-add-remo…
sea-snake 28e4640
Make OpenID credential in StableAnchor not optional.
sea-snake 04117db
Make OpenID credentials in IdentityAnchorInfo optional so it's backwa…
sea-snake 072252c
🤖 cargo-fmt auto-update
github-actions[bot] 80d94ac
Make OpenID credentials in IdentityAnchorInfo optional so it's backwa…
sea-snake 3042122
🤖 npm run generate auto-update
github-actions[bot] a555694
Fix principal from seed in tests
sea-snake bed886e
🤖 cargo-fmt auto-update
github-actions[bot] 114d254
Changes based on feedback and added tests for new `Anchor` methods.
sea-snake 2a6827b
🤖 cargo-fmt auto-update
github-actions[bot] 4639e4e
Fix serde name for archive operation.
sea-snake 5a7e8ef
Merge remote-tracking branch 'origin/sea-snake/openid-google-add-remo…
sea-snake aa1f722
Wrap tuple with struct since stable-structures doesn't directly suppo…
sea-snake 0049fbc
🤖 cargo-fmt auto-update
github-actions[bot] 5c47f82
Update based on feedback.
sea-snake 3c78479
Update based on feedback.
sea-snake 810af38
🤖 cargo-fmt auto-update
github-actions[bot] 7c82534
Update based on feedback.
sea-snake e27c0ab
Merge remote-tracking branch 'origin/sea-snake/openid-google-add-remo…
sea-snake 91a5432
🤖 cargo-fmt auto-update
github-actions[bot] 73a061c
Update based on feedback.
sea-snake 6280d3b
Merge remote-tracking branch 'origin/sea-snake/openid-google-add-remo…
sea-snake bb5b426
Update based on feedback.
sea-snake 5e458d3
Update based on feedback.
sea-snake 94026e7
Update based on feedback.
sea-snake 41be074
Make lookup map many-to-many
sea-snake 5157a1c
Make lookup map many-to-many
sea-snake 34a8b8f
🤖 cargo-fmt auto-update
github-actions[bot] 2234e72
🤖 npm run generate auto-update
github-actions[bot] 9f061b7
Make lookup map many-to-many
sea-snake cfe4ffd
Merge remote-tracking branch 'origin/sea-snake/openid-google-add-remo…
sea-snake efac984
Implement canister methods and link to frontend to add and remove Ope…
sea-snake 81874c3
Implement canister methods and link to frontend to add and remove Ope…
sea-snake 4799a0e
WIP
sea-snake e1eb9af
WIP
sea-snake 4257fad
Undo changes
sea-snake 51c573b
Undo changes
sea-snake 985939d
Add check to make sure OpenID credential is only added to a single an…
sea-snake 0a74291
WIP
sea-snake 18bb775
WIP
sea-snake 0631fe2
Undo changes
sea-snake 7f90e07
Undo changes
sea-snake 770168a
🤖 cargo-fmt auto-update
github-actions[bot] f045a05
Fix import
sea-snake bfa4f0c
Merge remote-tracking branch 'origin/sea-snake/openid-google-add-remo…
sea-snake f7614a0
Add InternalCanisterError to error enums similar to v2 api.
sea-snake e6aec3e
Fix did file.
sea-snake 8bfe986
🤖 npm run generate auto-update
github-actions[bot] 37340fb
Remove frontend env variable and use backend config instead.
sea-snake 08e0bb5
Merge remote-tracking branch 'origin/sea-snake/openid-google-add-remo…
sea-snake 6c27565
Merge branch 'main' into sea-snake/openid-google-add-remove-from-anchor
sea-snake 2268e5d
Remove `#[allow(unused)]` from methods that are now used.
sea-snake 223447f
Update translation labels based on feedback.
sea-snake abd4a04
Add TODO comment.
sea-snake 0a1a7d7
Add `should_register_openid_credential_only_for_a_single_anchor` test.
sea-snake 69a7ac7
🤖 cargo-fmt auto-update
github-actions[bot] 1b47942
Add `should_register_openid_credential_only_for_a_single_anchor` test.
sea-snake eb20b55
Merge remote-tracking branch 'origin/sea-snake/openid-google-add-remo…
sea-snake fcbef18
Move imports into test itself.
sea-snake 0da1a61
Fix test
sea-snake File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sea-snake marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sea-snake marked this conversation as resolved.
Show resolved
Hide resolved
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this our dev environment? Did you check with IT about the prod one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a dev environment I created, only works internally within DFINITY. This should be removed at a later point and replaced with instructions on creating a client id and configuring it with II.
Haven't gotten around to ask IT about the prod one yet, we'll need to also ask them to create others for our other environments.