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

More Phone country codes and NonCashType for contributions #79

Merged
merged 5 commits into from
Feb 23, 2023

Conversation

guymaness2
Copy link
Contributor

@guymaness2 guymaness2 commented Feb 23, 2023

Description

What does the change add or fix?

Added a few more supported phone country codes as well as support for NonCashType on imported contributions.


Release Notes

What does the change add or fix in a succinct statement that will be read by clients?
  • Added country code support for Costa Rica, India, Poland, and Ukraine.
  • Added support for non-cash type for contributions.
  • Updated email validation to use Person.Email RegularExpression custom property pattern.

Requested By

Who reported, requested, or paid for the change?

KFS


Screenshots

Does this update or add options to the block UI?

none


Change Log

What files does it affect?
  • Bulldozer.CSV/CSVComponent.cs
  • Bulldozer.CSV/CSVPhoneCountryCode.cs
  • Bulldozer.CSV/Maps/Financial.cs
  • Bulldozer.CSV/Maps/Individual.cs
  • Bulldozer/Utility/AddMethods.cs
  • Bulldozer/Utility/Extensions.cs

Migrations/External Impacts

Is it a breaking change for other versions/clients?

no

@nateh777 nateh777 merged commit 700446b into hotfix-14 Feb 23, 2023
@nateh777 nateh777 deleted the AddPhoneCountryCodes branch February 23, 2023 20:36
nateh777 added a commit that referenced this pull request Feb 23, 2023
More Phone country codes and NonCashType for contributions

(cherry picked from commit 700446b)
nateh777 added a commit that referenced this pull request Feb 23, 2023
More Phone country codes and NonCashType for contributions

(cherry picked from commit 700446b)
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.

3 participants