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

Delete text on user me loans #1024

Merged
merged 2 commits into from
Mar 15, 2024
Merged

Conversation

Adamik10
Copy link
Contributor

Link to issue

https://reload.atlassian.net/browse/DDFLSBP-463

Description

Removes subheadline from loan list modal about where the loan can be returned.

Screenshot of the result

Additional comments or questions

Base automatically changed from sprint/lsbp-12 to develop March 14, 2024 12:52
Copy link
Contributor

@Dresse Dresse left a comment

Choose a reason for hiding this comment

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

Good that you have removed it in dpl-react, but I also think it would be best to remove it from dpl-cms.

Also shouldn't it also be removed from the groupModalArgs.ts file?
https://github.com/danskernesdigitalebibliotek/dpl-react/blob/sprint/lsbp-12/src/core/storybook/groupModalArgs.ts

Copy link
Contributor

@Dresse Dresse left a comment

Choose a reason for hiding this comment

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

LGTM

@Adamik10
Copy link
Contributor Author

Jeez, thx for this!
I am a bit slow when it comes to thinking these days 😅

@Dresse
Copy link
Contributor

Dresse commented Mar 15, 2024

Jeez, thx for this! I am a bit slow when it comes to thinking these days 😅

No worries! 😄

@Adamik10 Adamik10 changed the base branch from develop to release/brahma-13 March 15, 2024 13:18
@Adamik10 Adamik10 merged commit 3f3b4cd into release/brahma-13 Mar 15, 2024
22 checks passed
@Adamik10 Adamik10 deleted the delete-text-on-user-me-loans branch March 15, 2024 13:18
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.

4 participants