Skip to content

Commit

Permalink
Merge branch 'main' into lmendoza/remove-firebase
Browse files Browse the repository at this point in the history
  • Loading branch information
leomendoza123 authored Nov 20, 2024
2 parents 1c78ba9 + 03c6652 commit 8322647
Show file tree
Hide file tree
Showing 6 changed files with 89 additions and 63 deletions.
9 changes: 0 additions & 9 deletions .github/workflows/bld_test_rel_tag.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,21 +81,13 @@ jobs:
- format_i18n
- format_prettier

# this is a test
bld_yarn:
uses: ./.github/workflows/bld_yarn.yml
needs:
- format_i18n
- format_prettier

##############################################################################
# uses maven to build via yarn into a war file
bld_mvn:
uses: ./.github/workflows/bld_mvn.yml
secrets: inherit # pass all secrets for uploading assets
needs:
- lint
- bld_yarn
- format_i18n
- format_prettier
permissions:
Expand All @@ -113,7 +105,6 @@ jobs:
secrets: inherit # pass all secrets for uploading assets
needs:
- lint
- bld_yarn
- format_i18n
- format_prettier
permissions:
Expand Down
8 changes: 5 additions & 3 deletions .github/workflows/bld_yarn.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,12 @@ jobs:
matrix:
include:
- artifact_name: prod
- artifact_name: sandbox
- artifact_name: qa
- artifact_name: int
# yarn build is only used as a test
# - artifact_name: sandbox
# - artifact_name: qa
# - artifact_name: int
runs-on: ubuntu-latest
timeout-minutes: 15
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
Expand Down
8 changes: 8 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,11 @@
## v2.107.13 - 2024-11-18

[Full Changelog](https://github.com/ORCID/orcid-angular/compare/v2.107.12...v2.107.13)

## v2.107.12 - 2024-11-15

[Full Changelog](https://github.com/ORCID/orcid-angular/compare/v2.107.11...v2.107.12)

## v2.107.11 - 2024-11-13

[Full Changelog](https://github.com/ORCID/orcid-angular/compare/v2.107.10...v2.107.11)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
width: 100%;
display: flex;
justify-content: center;
height: 100vh;
}

main.stand-alone-mode {
Expand Down
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

0 comments on commit 8322647

Please sign in to comment.