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

allow embago and leased works to be updated #4021

Merged
merged 1 commit into from
Sep 20, 2019
Merged

allow embago and leased works to be updated #4021

merged 1 commit into from
Sep 20, 2019

Conversation

blancoj
Copy link
Contributor

@blancoj blancoj commented Sep 19, 2019

Fixes #3958
Allows embargoed and leased works to be updated. Before this change, the only way an embargoed or leased work could be updated was if the date range were changed. I have placed an explanatory comment on how to get around this in the code.

Guidance for testing, such as acceptance criteria or new user interface behaviors:

  • Create an embargo work.
  • Edit the title of the work and save it.
  • Verify the title was changed.
  • Edit the work again, and this time add a file to the work and save it.
  • Verify that the new file was uploaded.
    AND
  • Create a leased work.
  • Edit the title of the work and save it.
  • Verify the title was changed.
  • Edit the work again, and this time add a file to the work and save it.
  • Verify that the new file was uploaded.

@samvera/hyrax-code-reviewers

@blancoj blancoj changed the title [WIP] allow embago and leased wokrs to be editted allow embago and leased wokrs to be editted Sep 19, 2019
@blancoj blancoj changed the title allow embago and leased wokrs to be editted allow embago and leased wokrs to be updated Sep 19, 2019
@blancoj blancoj changed the title allow embago and leased wokrs to be updated allow embago and leased works to be updated Sep 19, 2019
Copy link
Contributor

@no-reply no-reply left a comment

Choose a reason for hiding this comment

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

Thanks! Nice fix!

@no-reply no-reply merged commit b2d8cd6 into master Sep 20, 2019
@no-reply no-reply deleted the issue-3958 branch September 20, 2019 18:14
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.

Unable to Add Files to Work under an Embargo
2 participants