Display author description on author merge page #9453
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.
Implement Author Description on Merge Page
Closes #9429
Description
This PR introduces the display of author descriptions on the merge page. The author descriptions will provide more context about each author during the merge process, improving the usability and user experience and helping librarians disambiguate authors, and avoid accidentally merging authors that shouldn't be merged.
Changes
Conditional Display of Author Description:
TruncateString
macro to limit the displayed author description to 250 characters for better readability and to ensure a consistent layout.Updated Template:
How to Test
Run the Application:
Ensure the application is running locally.
Navigate to the Merge Page:
Go to the merge page where authors are listed for merging.
Verify Author Descriptions:
Check for Errors:
Ensure there are no errors or layout issues introduced by these changes.
Related Issues
Notes
TruncateString
macro to ensure descriptions are concise and fit within the layout.Checklist
Additional Comments
Please review the changes and provide feedback. If there are any questions or further modifications required, let me know. Thank you!