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

Iteration 5 #479

Merged
merged 12 commits into from
Dec 15, 2023
Merged

Iteration 5 #479

merged 12 commits into from
Dec 15, 2023

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

milanmajchrak and others added 12 commits November 15, 2023 12:29
* The admin could download restricted bitstreams.

* The user cannot download bitstream if he/she is not signed in.

* Cannot obtain context with user.

* The user is already fetched from the context.

* Added docs.

* Do not use endpoint for downloading the single file because it already exists - remove it.

* Fixed AuthorizationRestControllerIT tests.
* Find bitstream format using mimetype not ID.

* Updated tests according to fix.
* Updated `handle.canonical.prefix` to hdl handle.

* Integration test cannot have changed handle.canonical.prefix.
* The user registration is added when the eperson is created by the ui.

* try using newer checkout, same as upstream, since error is with git

---------

Co-authored-by: MajoBerger <[email protected]>
* Add logging of downloading restricted bitstreams.

* Log downloading of every bitstream not only restricted ones.

* Added docs

* Refactored logging.
* givenname and last name values are converted to UTF-8 encoding

* Delete eperson in tests
Fixed emails sending - added arguments and cfg properties that are sent in the email. (#470)
* DSpace#8585 Add submitter information to provenance metadata

* Cherry picked fix from vanilla and added test to check if the Item provenance metadata is added

---------

Co-authored-by: Adán Román Ruiz <[email protected]>
* Created endpoint for fetching IP address with tests.

* Made the code more understable
* fixes ufal#1096 (#471)

* Added publisher filter in the integration test check

---------

Co-authored-by: Ondřej Košarko <[email protected]>
@milanmajchrak milanmajchrak changed the title Customer/uk it 5 Iteration 5 Dec 15, 2023
@milanmajchrak milanmajchrak self-assigned this Dec 15, 2023
@milanmajchrak milanmajchrak merged commit 3d88a75 into customer/uk Dec 15, 2023
2 of 4 checks passed
@MajoBerger MajoBerger deleted the customer/uk-it-5 branch August 5, 2024 10:06
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.

2 participants