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

set is_deceased to not null in schema and upgrade script dev/core#1697 #17025

Conversation

michaelmcandrew
Copy link
Contributor

@civibot
Copy link

civibot bot commented Apr 8, 2020

(Standard links)

@civibot civibot bot added the master label Apr 8, 2020
@colemanw colemanw force-pushed the 1697-set-is_deceased-to-not-null-in-schema-and-upgrade-script branch from ecf2d18 to 2cd5d41 Compare April 9, 2020 13:01
@colemanw
Copy link
Member

colemanw commented Apr 9, 2020

Hey @michaelmcandrew - thanks for this, good to get the schema cleaned up.

Two FYIs:

  1. Updating a schema xml file requires running /xml/GenCode.php to regenerate the corresponding DAO file.
  2. Please avoid creating branches on github/civicrm and use your own fork to keep things tidy.

Cheers!

@colemanw
Copy link
Member

colemanw commented Apr 9, 2020

I've done the regen and also squashed the commits. This is now good to go!

@seamuslee001 seamuslee001 merged commit 828c4c6 into master Apr 9, 2020
@seamuslee001 seamuslee001 deleted the 1697-set-is_deceased-to-not-null-in-schema-and-upgrade-script branch April 9, 2020 20:41
@michaelmcandrew
Copy link
Contributor Author

  1. Updating a schema xml file requires running /xml/GenCode.php to regenerate the corresponding DAO file.

Thanks - should have remembered to do that.

  1. Please avoid creating branches on github/civicrm and use your own fork to keep things tidy.

Sorry - I keep on doing that - will try harder in future.

@colemanw
Copy link
Member

No worries thanks for the PR @michaelmcandrew

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

Successfully merging this pull request may close these issues.

4 participants