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

add api t&c panel to dev tools #2397

Merged
merged 2 commits into from
Nov 18, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -204,56 +204,47 @@ <h2 class="orc-font-body-large" i18n="@@developerTools.redirectUris">
your integration users will experience an error.
</p>

<div class="info">
<div class="col">
<mat-icon class="material-icons-outlined large-material-icon"
>info</mat-icon
>
</div>
<div class="col l11 m6 s3">
<strong i18n="@@developerTools.pleaseNote">Please note</strong>
<ul>
<li>
<ng-container i18n="@@developerTools.only">
Only</ng-container
>
<strong i18n="@@developerTools.httpsURI">HTTPS URIs</strong>
<ng-container i18n="@@developerTools.areAccepted">
are accepted in production
</ng-container>
</li>
<li>
<ng-container i18n="@@developerTools.domainsRegisted"
>Domains registered</ng-container
>
<strong i18n="@@developerTools.must">MUST</strong>
<ng-container i18n="@@developerTools.exactlyMatchTheDomains"
>exactly match the domains used, including
subdomains</ng-container
>
</li>
<li>
<strong i18n="@@developerTools.registerAllRedirectURIs"
>Register all redirect URIs fully where possible.</strong
>
<ng-container i18n="@@developerTools.thisIsTheMostSecure"
>This is the most secure option and what we recommend. For
more information about redirect URIs, please see
our</ng-container
>

<a
class="underline"
i18n="@@developerTools.redirectUriFaq"
rel="noopener noreferrer"
target="_blank"
href="https://info.orcid.org/ufaqs/how-do-redirect-uris-work/"
>redirect URI FAQ</a
>
</li>
</ul>
</div>
</div>
<app-alert-message class="redirect-uri-notice">
<strong i18n="@@developerTools.pleaseNote">Please note</strong>
<ul>
<li>
<ng-container i18n="@@developerTools.only"> Only</ng-container>
<strong i18n="@@developerTools.httpsURI">HTTPS URIs</strong>
<ng-container i18n="@@developerTools.areAccepted">
are accepted in production
</ng-container>
</li>
<li>
<ng-container i18n="@@developerTools.domainsRegisted"
>Domains registered</ng-container
>
<strong i18n="@@developerTools.must">MUST</strong>
<ng-container i18n="@@developerTools.exactlyMatchTheDomains"
>exactly match the domains used, including
subdomains</ng-container
>
</li>
<li>
<strong i18n="@@developerTools.registerAllRedirectURIs"
>Register all redirect URIs fully where possible.</strong
>
<ng-container i18n="@@developerTools.thisIsTheMostSecure"
>This is the most secure option and what we recommend. For
more information about redirect URIs, please see
our</ng-container
>

<a
class="underline"
i18n="@@developerTools.redirectUriFaq"
rel="noopener noreferrer"
target="_blank"
href="https://info.orcid.org/ufaqs/how-do-redirect-uris-work/"
>redirect URI FAQ</a
>
</li>
</ul>
</app-alert-message>

<div class="warn" *ngIf="triedToSaveWithoutUrls">
<div class="col">
Expand Down Expand Up @@ -453,6 +444,27 @@ <h3 class="orc-font-body" i18n="@@developerTools.exampleCode">
Redirect URIs must be unique
</mat-error>
</p>
<app-alert-message class="notice-panel"
><ng-container
i18n="
@@developerTools.theOrcidPublicApiAllowsYouToRequestPermission"
>The ORCID Public API is free for non-commercial use by
individuals as stated in the</ng-container
>
<a
href="https://info.orcid.org/public-client-terms-of-service/"
target="_blank"
class="underline"
i18n="@@developerTools.publicClientTermsOfService"
>Public APIs Terms of Service.</a
>
<ng-container i18n="@@developerTools.byRegisteringForPublicApi"
>By “non-commercial” we mean that you may not charge any re-use
fees for the Public API, and you may not make use of the Public
API in connection with any revenue-generating product or
service.</ng-container
></app-alert-message
>

<button
mat-raised-button
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,7 @@ p {

ul {
padding-left: 16px;
margin-bottom: 0;
}

.add-more {
Expand Down Expand Up @@ -148,6 +149,19 @@ app-client-secret {
margin-top: 16px;
}

.notice-panel {
line-height: 21px;
margin-top: 32px;
}

.redirect-uri-notice {
strong {
display: inline;
}
line-height: 21px;
margin-bottom: 32px;
}

.save-button {
margin-top: 32px;
margin-bottom: 32px;
Expand Down