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

pass-ember CMF test candidate #230

Closed
wants to merge 2 commits into from

Conversation

jabrah
Copy link
Contributor

@jabrah jabrah commented Jul 4, 2019

Bump version number of pass-ember to a candidate for testing.

Some issues still exist in this image, but the holiday fast approaches and I wanted to get something out for people to look at. If we'd rather wait for other fixes to come it before testing, then we could always close this PR in favor of a newer one.

Highlights

Select changes since the last version bump (~3 weeks ago):

  • When a user clicks the "Delete" button on a draft submission, they now have their status changed to cancelled instead of being fully deleted in Fedora to get around some nasty permission issues
  • Phantom draft submissions shouldn't appear in the submissions table anymore. Addressed a timing issue between "deleting" a draft submission and ES indexing that was causing deleted drafts to appear in the submissions table
  • Made metadata loaded from the DOI read only again in the metadata forms
  • Force DOI to be loaded when a user enters the submission workflow to address a few data consistency issues
  • Fixed bug where metadata from the DOI would not be cleared from the metadata forms when a user clears the DOI from the publication
  • Map Crossref field issued (date) to publicationDate in the metadata blob
  • Use an upgraded schema service to show a merged metadata form when possible. If a 409 error is seen from the service, the UI will try to get the unmerged forms to show the user
  • Display co-PIs better in grants details page
  • Force grants table and grant details page to ignore "deleted" submissions

Known Issues

There are still some major bugs known to be present in this image including:

@jabrah jabrah added the CMF label Jul 4, 2019
@htpvu htpvu requested a review from markpatton July 15, 2019 17:47
@markpatton markpatton closed this Jul 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants