Made ConfirmationModalComponent more generic #2338
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
References
Description
Currently the
ConfirmationModalComponent
uses the name from theDSpaceObject
s to render the labels. The problem with this approach is that this modal only works for DSpace object and if you would want to use it forMetadataField
(or any other child ofListableObject
which isn't a child ofDSpaceObject
) it wouldn't work. Therefore it's better to only give the name to the modal instead of the wholeDSpaceObject
object.Instructions for Reviewers
To make this component more generic I replaced the
@Input() dso: DSpaceObject;
with@Input() name: string
and replaced all the usagesGuidance for how to test this PR:
Checklist
yarn lint
yarn check-circ-deps
)package.json
), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.