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

4438 - Button Remain Activated After Selecting the Storage #4485

Merged

Conversation

Naraveni
Copy link
Contributor

@Naraveni Naraveni commented Jun 29, 2024

Resolves #4438

Description

Add Item button was getting de-activated if storage location is selected after first attempt of submitting form without selecting anything.

Include anything else we should know about. -->

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@Naraveni Naraveni force-pushed the 4438-ACTIVATE-ADD-ANOTHER-ITEM-AFTER-ERROR branch 3 times, most recently from 947a85e to e7f99ca Compare June 29, 2024 16:16
cielf
cielf previously requested changes Jun 29, 2024
Copy link
Collaborator

@cielf cielf left a comment

Choose a reason for hiding this comment

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

Hey @Naraveni Thanks for this -- I checked it out manually, and it works for distributions.

The issue also asked for tests confirming that the behaviour works. Please provide those.

Thanks again!

@Naraveni Naraveni force-pushed the 4438-ACTIVATE-ADD-ANOTHER-ITEM-AFTER-ERROR branch from e7f99ca to da2b1ac Compare July 1, 2024 02:53
@Naraveni
Copy link
Contributor Author

Naraveni commented Jul 1, 2024

I have added test in distrbution and transfers system specs files as the changes I made are in js file which is related to distributions and transfers as the file name suggests.

@Naraveni Naraveni force-pushed the 4438-ACTIVATE-ADD-ANOTHER-ITEM-AFTER-ERROR branch from da2b1ac to 85e004a Compare July 1, 2024 02:55
@Naraveni Naraveni requested a review from cielf July 1, 2024 02:55
@cielf cielf dismissed their stale review July 2, 2024 17:44

addressed

@cielf
Copy link
Collaborator

cielf commented Jul 2, 2024

Functionality checks out. Over to @dorner for a technical pass

@cielf cielf requested a review from dorner July 2, 2024 17:46
spec/system/distribution_system_spec.rb Outdated Show resolved Hide resolved
spec/system/transfer_system_spec.rb Show resolved Hide resolved
@Naraveni Naraveni force-pushed the 4438-ACTIVATE-ADD-ANOTHER-ITEM-AFTER-ERROR branch 2 times, most recently from 0f4ca3e to 0649768 Compare July 13, 2024 12:59
@Naraveni Naraveni force-pushed the 4438-ACTIVATE-ADD-ANOTHER-ITEM-AFTER-ERROR branch from 0649768 to 24a6d0f Compare July 13, 2024 13:09
@Naraveni
Copy link
Contributor Author

And also the error message in transfers creation page was coming wrong ias the
tags were not treated as html and included in normal text. I though of using html_safe but it is a potential risk, So I Simple went with (, ) separator which we are doing in lot of place in our code base

@Naraveni Naraveni requested a review from dorner July 13, 2024 13:12
Copy link
Collaborator

@cielf cielf left a comment

Choose a reason for hiding this comment

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

LGTM also

@cielf cielf merged commit f747006 into rubyforgood:main Jul 15, 2024
19 checks passed
Copy link
Contributor

@Naraveni: Your PR 4438 - Button Remain Activated After Selecting the Storage is part of today's Human Essentials production release: 2024.07.21.
Thank you very much for your contribution!

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.

Distribution, New. Add Another Item should be activated when storage location chosen, after error.
3 participants