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

Remove permissions for usAddressVerification #6598

Merged
merged 1 commit into from
Oct 30, 2023

Conversation

brianteeman
Copy link
Contributor

Missed from #6585

Note I do not know if you normally create an update sql for this and if the demo.sql should be updated as well

What does PR do? Any background context you want to provide?

What Issues does it Close? Any the relevant tickets?

Closes

Screenshots (if appropriate)

Where should the reviewer start?

How should this be manually tested?

Questions:

  • Is there a related website / article to substantiate / explain this change?

    • Yes - Link:
    • No
  • Does the development wiki need an update?

    • Yes - Link:
    • No
  • Does the user documentation wiki need an update?

    • Yes - Link:
    • No
  • Does this add new dependencies?

    • Yes
    • No
  • Does this need to add new data to the demo database

    • Yes
    • No

Missed from ChurchCRM#6585

Note I do not know if you normally create an update sql for this and if the demo.sql should be updated as well
@DawoudIO DawoudIO added this to the 5.0.5 milestone Oct 30, 2023
@DawoudIO
Copy link
Contributor

Ya I may switch 5.0.5 to 5.1.0 and do a db cleanup

@DawoudIO DawoudIO merged commit 1c5f41c into ChurchCRM:master Oct 30, 2023
3 checks passed
@brianteeman brianteeman deleted the usAddressVerification branch October 30, 2023 15:02
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.

2 participants