Skip to content

Fix "null" and unpopulated fields#58

Open
Lightning11wins wants to merge 4 commits into
masterfrom
import-partner-fix
Open

Fix "null" and unpopulated fields#58
Lightning11wins wants to merge 4 commits into
masterfrom
import-partner-fix

Conversation

@Lightning11wins
Copy link
Copy Markdown

@Lightning11wins Lightning11wins commented Apr 28, 2026

Fix blank fields being populated with the word null and non-blank fields not being populated when adding new partners during a gift import from the gift import app.

Records in the new partner app before this change:
image

Records in the new partner app after this change:
image

@Lightning11wins Lightning11wins requested review from maasj and nboard April 28, 2026 18:22
@Lightning11wins Lightning11wins self-assigned this Apr 28, 2026
@Lightning11wins Lightning11wins added bug size: trivial Easy to review, probably ~100 lines or fewer. labels Apr 28, 2026
nboard
nboard previously requested changes Apr 28, 2026
Copy link
Copy Markdown
Contributor

@nboard nboard left a comment

Choose a reason for hiding this comment

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

Look great! Just a couple of potential problems with phone country codes to review.

Comment thread kardia-app/modules/base/new_partner.cmp
Comment thread kardia-app/modules/base/new_partner.cmp Outdated
Fix missing set_phone_area_country parameter.
Rename set_phone_country_cn to set_phone_area_country_guess_cn to improve clarity.
Fix set_phone_area_country_guess_cn triggering even if a correct phone area country code was explicitly specified.
@Lightning11wins Lightning11wins requested a review from nboard April 28, 2026 22:50
@nboard
Copy link
Copy Markdown
Contributor

nboard commented Apr 28, 2026

Looks good to me!
@gbeeley Could you take a look at this to approve merging when you get the chance?

@nboard nboard dismissed their stale review April 28, 2026 22:57

The necessary changes were made

@Lightning11wins
Copy link
Copy Markdown
Author

Lightning11wins commented Apr 28, 2026

Great, are you allowed to approve the PR so that I can merge it or do we need to wait for Greg to review and approve it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug size: trivial Easy to review, probably ~100 lines or fewer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants