-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat(backend): Gate api methods #1807
Changes from 22 commits
ba2aa51
60809b7
b2e0d20
491eb6a
7a8c793
5fde54c
235c2ad
54637a6
4791ed8
19b07d2
306f963
82a7aa7
c90a833
e077aa8
2cdd8d5
c5708af
0dc956c
458ce9b
270a498
1dd3355
42c8d9a
f479d50
c6ca55e
d8b56b9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -108,6 +108,7 @@ pub(crate) fn init_arg() -> Arg { | |
issuer_origin: ISSUER_ORIGIN.to_string(), | ||
credential_type: CredentialType::ProofOfUniqueness, | ||
}]), | ||
api: None, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @bitdivine: Will you add some integration tests that exercise the API disabling? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Disabling APIs matters in the migration, so I was going to test that in the migration e2e tests. After the migration is complete I expect all of this will be deleted again. 😄 |
||
}) | ||
} | ||
|
||
|
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.
We wish to be able to make features either fully enabled, fully disabled, or enabled for read methods only. This type is used to represent which of these states a feature is in.