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

customer/uk-it-7 #436

Merged
merged 24 commits into from
Jan 3, 2024
Merged

customer/uk-it-7 #436

merged 24 commits into from
Jan 3, 2024

Conversation

milanmajchrak
Copy link
Collaborator

Phases MP MM MB MR JM Total
ETA 0 0 0 0 0 0
Developing 0 0 0 0 0 0
Review 0 0 0 0 0 0
Total - - - - - 0
ETA est. 0
ETA cust. - - - - - 0

Problem description

MajoBerger and others added 24 commits November 16, 2023 13:28
Run import from main branch
Added missing tranlates and updated messages after running the script
Redirect to login with `redirectUrl` param because it is lost after click on `local` login button. (#404)
* Added static pages from the lindat git

* Fixed redirection and created safeHtml pipe.

* Could redirect out of namespace url, small refactoring.

* Updated tests.

* Added images into the static pages.
* Encoded query params (#407)
* The clients IP address is fetched from the BE API.

* Replace LegacyBitstreamUrlResolver by BitstreamBreadcrumbResolver because the first one throws errors to the console.
* Fixed static page redirects.

* Refactored processing links.
* Separated toggling of DiscoJuice popup from the workign with redirect URL

* The downloading page is loaded before downloading.

* Login redirect works.

* Removed storing cookies in the aai.js
* If the file exceeds the upload max file size the uploading will be stopped before starting and the user will see proper error message.

* Fixed unit tests - added configurationDataService
* Do not show licenses if the Item doesn't have any file.
* ClarinAuthorization passed, but vanilla not because of vanilla check - I added dtoken into vanilla authorization url and because of that the vanilla authorization will be passed.

* Added message for the expiration token message - it was hardcoded.
* Loaded property from the cfg and check if the page with idp attributes could be showed up. If not redirect the user to the home page. (#431)
* The admin could see in the bitstream admin UI if the bitstream is stored in both storages (S3, local).

* Fixed failing unit test - updated columnSizes object
Added curate collection edit translation and curation task name. (#434)
* Moved file size limit into FileUploader options and handled Upload cancelation.

* Check that uploading is successful
* BitstreamChecksum values are fetched and parsed from the BE

* Checksum info is showed up.

* Added messages and translations.

* Added docs and refactored code

* Fixed failing tests

* Fixed wrong czech translations.
* Send redirectUrl param to the IdP in the target url

* The code is updated to be more readable.
@milanmajchrak milanmajchrak self-assigned this Jan 3, 2024
@milanmajchrak milanmajchrak merged commit f1c5545 into customer/uk Jan 3, 2024
4 of 6 checks passed
@MajoBerger MajoBerger deleted the customer/uk-it-7 branch March 4, 2024 07:26
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