-
Notifications
You must be signed in to change notification settings - Fork 16
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
Updated .NET samples #1353
Updated .NET samples #1353
Conversation
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.
Overall, I approve of the code snippets. Let's not conflate the removal of deprecated endpoints from the docs with code snippet creation.
## RetrieveDomainVerificationRecord | ||
|
||
Retrieve a random hash TXT that can be used to verify domain ownership | ||
|
||
{{ proto_sample_start("services.provider.v1.Provider.RetrieveDomainVerificationRecord") }} | ||
{{ proto_sample_code("services.provider.v1.Provider.RetrieveDomainVerificationRecord") }} | ||
{{ proto_method_tabs("services.provider.v1.Provider.RetrieveDomainVerificationRecord") }} | ||
|
||
--- | ||
|
||
## RefreshDomainVerificationStatus | ||
|
||
Call to verify domain | ||
|
||
{{ proto_sample_start("services.provider.v1.Provider.RefreshDomainVerificationStatus") }} | ||
{{ proto_sample_code("services.provider.v1.Provider.RefreshDomainVerificationStatus") }} | ||
{{ proto_method_tabs("services.provider.v1.Provider.RefreshDomainVerificationStatus") }} | ||
|
||
--- | ||
|
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.
Why are these removed?
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.
My understanding is we had agreed at some point that these should not be part of the public-facing SDK but rather the job of the Dashboard -- I can add samples back in if we like but I'd prefer to move forward with porting them out of the SDK.
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.
Agreed. Let's remove these.
<!-- | ||
// This call is not yet implemented | ||
## List Ecosystems | ||
|
||
Lists all available ecosystem for the current authentication context. | ||
|
||
When using one of the SDKs, you must supply an [List Ecosystem Request](../proto/index.md#listecosystemrequest) object. This object follows the model below: | ||
|
||
{{ proto_message('services.provider.v1.ListEcosystemRequest') }} | ||
|
||
The response model is of type [List Ecosystem Response](../proto/index.md#listecosystemresponse): | ||
|
||
{{ proto_message('services.provider.v1.ListEcosystemResponse') }} | ||
--> | ||
|
||
## Invite | ||
|
||
This will be removed May 1, 2023 | ||
Invite a user to the ecosystem | ||
|
||
{{ proto_sample_start("services.provider.v1.Provider.Invite") }} | ||
{{ proto_sample_code("services.provider.v1.Provider.Invite") }} | ||
{{ proto_method_tabs("services.provider.v1.Provider.Invite") }} | ||
|
||
--- | ||
|
||
## InvitationStatus | ||
|
||
This will be removed May 1, 2023 | ||
Check the status of an invitation | ||
|
||
{{ proto_sample_start("services.provider.v1.Provider.InvitationStatus") }} | ||
{{ proto_sample_code("services.provider.v1.Provider.InvitationStatus") }} | ||
{{ proto_method_tabs("services.provider.v1.Provider.InvitationStatus") }} | ||
|
||
--- |
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.
Do we want to remove these yet?
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.
These haven't been called a single time in the past month and we plan to remove them shortly -- might as well bite the bullet and not perform the work of documenting endpoints that are effectively already dead.
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.
@janpieterz thoughts?
Fair -- I'll make a separate PR and the discussion can live there |
Moving to draft as we are pausing this work |
Bump @geel9 @fundthmcalculus . We still want this in or can it be closed? |
SearchRegistry
sample -- @tmarkovski please advise (current samples don't include an example query)my-ecosystem
An Example Credential
/an-example-credential
/urn:template:my-ecosystem:an-example-credential
firstName
,lastName
,age
[email protected]
urn:uuid:7afd0191-9656-3a51-c12f-44ecfbb5a829
urn:revocation-registry:my-ecosystem:AD9wKZCr4zVxat1XSawScD#42
urn:egf:my-ecosystem:1297e60abb3b425faaaf0f0a551aae2c