Skip to content

4976 enable defaulting to contact name#5163

Merged
dorner merged 3 commits into
rubyforgood:mainfrom
bsbonus:4976-enable-defaulting-to-contact-name
May 2, 2025
Merged

4976 enable defaulting to contact name#5163
dorner merged 3 commits into
rubyforgood:mainfrom
bsbonus:4976-enable-defaulting-to-contact-name

Conversation

@bsbonus
Copy link
Copy Markdown
Contributor

@bsbonus bsbonus commented Apr 25, 2025

Resolves #4976

Description

The UI was incorrectly rendering null/empty for Product Drive Participant names during the Donation creation form, if the Participant had not included a business name

Type of change

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

How Has This Been Tested?

Repro:
Sign in as org_admin1@example.com
click on Community, then Product Drive Participant in the left-hand-menu
click on New Product Drive Participant
fill in a contact name and an email - leave business name blank
save
click on Donations, then New Donation in the left-hand menu
Choose Product Drive as source
pick a product drive
click on the product drive participant field.
You will see that there is a blank in the dropdown for the participant you entered.

Screenshots

Screenshot 2025-04-25 at 3 51 30 PM after_fix before_fix

@bsbonus bsbonus force-pushed the 4976-enable-defaulting-to-contact-name branch from d97549c to d15baae Compare April 25, 2025 22:59
@cielf cielf self-requested a review April 26, 2025 02:03
cielf
cielf previously requested changes Apr 26, 2025
Copy link
Copy Markdown
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.

@bsbonus Thanks for this!

There is something amiss, still -- I expect it wasn't described in the issue, because it wasn't obvious in the user interface that there is a difference until now.

There are two ways that you can enter a product drive participant -- either by filling in the form in the "Product Drive Participants" section under "Community" -- which is how we described the issue, or by entering it 'on the fly" by selecting "create new participant" in the Product Drive Participant field in a new Donation.

If you use the second method to enter a participant, any participant that doesn't have a business name is appearing as blank in the resultant dropdown.

(They'll appear initially, but not after the entry. If you enter one without a business name, it appears as blank.)

Can you take a look at that aspect, please? Thanks.

@bsbonus bsbonus requested a review from cielf April 26, 2025 17:54
@bsbonus
Copy link
Copy Markdown
Contributor Author

bsbonus commented Apr 26, 2025

@cielf yep -- same problem. Good catch, I don't know the app well enough to consider additional flows like that. PR is up for re-review

@cielf cielf dismissed their stale review April 28, 2025 16:08

Addressed

@cielf
Copy link
Copy Markdown
Collaborator

cielf commented Apr 28, 2025

@bsbonus -- this looks good functionality wise, but it does look llike the lint is failing. Flagging @dorner for input on the technical side.

@cielf cielf requested a review from dorner April 28, 2025 16:13
@bsbonus
Copy link
Copy Markdown
Contributor Author

bsbonus commented Apr 28, 2025

My bad, I thought the linter was setup to run on save in my ide. Should be sorted

@dorner
Copy link
Copy Markdown
Collaborator

dorner commented May 2, 2025

All good here!

@dorner dorner merged commit fc95a66 into rubyforgood:main May 2, 2025
11 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2025

@bsbonus: Your PR 4976 enable defaulting to contact name is part of today's Human Essentials production release: 2025.05.04.
Thank you very much for your contribution!

GiovannyCordeiro pushed a commit to GiovannyCordeiro/human-essentials that referenced this pull request Jun 6, 2025
* bug-4976 Defaults dropdown menu to use contact name when no business name is present

* bug-4976 updates rendering for contact-name & business name when creating a drive participant 'on-the-fly'

* Fixes silly linter mistake in spec file

---------

Co-authored-by: Brian Bonus <bonus@MacBookPro.lan>
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.

Donation - dropdown for product drive participants should show contact name if no business name given.

3 participants