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

feat(feature-details): open secure docs and images from depot API #1015

Merged
merged 22 commits into from
May 16, 2022

Conversation

LAMM26
Copy link
Contributor

@LAMM26 LAMM26 commented Mar 18, 2022

Please check if the PR fulfills these requirements

What is the current behavior? (You can also link to an open issue here)
URLs to external docs or images present in the getInfo table can sometimes be stored in the depot API and those are currently not validated by the app (unlike the user guides).

What is the new behavior?
Docs and images stored in the depot API can now be accessed in the getInfo table if the user has the credentials to access them.
Other small typo corrections

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications:

Other information:

@PhilippeLafreniere18 PhilippeLafreniere18 marked this pull request as draft March 18, 2022 16:13
@PhilippeLafreniere18 PhilippeLafreniere18 marked this pull request as ready for review April 6, 2022 14:08
Copy link
Member

@pelord pelord left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just review why this is hardcoded

url = url.match(regexDepot)[0];

@pelord
Copy link
Member

pelord commented May 2, 2022

@LAMM26 Please resolve merge conflicts! Try to merge 1.11.1

@LAMM26
Copy link
Contributor Author

LAMM26 commented May 3, 2022

@LAMM26 Please resolve merge conflicts! Try to merge 1.11.1

Should be resolved

@pelord
Copy link
Member

pelord commented May 16, 2022

@LAMM26 Try merge next into your branch please. Still some conflicts.

@PhilippeLafreniere18 PhilippeLafreniere18 merged commit 05cb02e into next May 16, 2022
@pelord pelord deleted the metadata_secure_link branch May 30, 2022 15:52
cbourget pushed a commit that referenced this pull request Mar 21, 2023
)

* feat(feature-details): verify credentials for urls in storage API

* refactor(feature-details): some optimizations

* fix(feature-details): small fixes

* refactor(feature-details): fix tags and translations

* fix(feature-details): fix logic

* fix(feature-details): slice storage url

* fixed regex and added click function

* changed api regex to config

* merge next

* fix conflict

* repaired bug with strings

* configService optional

* fix(query): Point with htmlgml2 were not rendered correctly.

Co-authored-by: Pierre-Etienne Lord <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants