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

DS-3937 Add support for styling and rows in the submission-forms.xml #38

Closed
wants to merge 4 commits into from

Conversation

abollini
Copy link
Member

This PR is the first part of DSpace#2103
It includes:

  • add row tag in the submissionform-definition so to allow to put multiple fields in the same row in the UI
  • add a style tag to provide additional information (as for instance extra css classes) to apply to the field in the UI

@coveralls
Copy link

coveralls commented Sep 13, 2018

Coverage Status

Coverage increased (+0.04%) to 28.933% when pulling d794c20 on inputform-rows into b6a35eb on DS-3580_workspaceitem.

@benbosman
Copy link

I have 2 questions for this PR:

  • How can users migrate to the new submission-forms.xml format? I don't see a script to convert their existing submission forms
  • Can you add Integration Tests for the customizations?

Copy link

@tdonohue tdonohue left a comment

Choose a reason for hiding this comment

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

Code looks good overall. Added one inline comment. Otherwise, it just needs Integration Tests.

@@ -639,4 +700,8 @@ public String getInputFormNameByCollectionAndField(Collection collection, String
}
throw new DCInputsReaderException("No field configuration found!");
}

public Map<String, List<String>> getMappedValuePairs() {
Copy link

Choose a reason for hiding this comment

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

Please add Javadocs to describe the structure of the "MappedValuePairs"

Copy link
Member Author

Choose a reason for hiding this comment

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

good catch! at the end it was an unused caching structure introduced in an early stage of the implementation... I have removed it

@abollini
Copy link
Member Author

abollini commented Oct 10, 2018

merged with the latest code in DSpace#2187 added some javadoc and the Integration Test

@abollini
Copy link
Member Author

move to the official DSpace repository

@abollini abollini closed this Oct 11, 2018
abollini pushed a commit that referenced this pull request Mar 3, 2020
[CST-2428] Added Advised Theses box, fix css rule
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.

None yet

4 participants