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

If somebody has no plan to upgrade a module - we should not block #280

Merged
merged 2 commits into from
Feb 28, 2021

Conversation

hamrant
Copy link

@hamrant hamrant commented Feb 26, 2021

Related Issue/Ticket:

#278 (review)

Steps to test:

  • Check that migration of event demo content does not fail with error

Quality checks:

Please check these boxes to confirm this PR covers the following cases:

  • Maintaining our upgrade path is essential. Check one or the other:
    • This PR provides updates via hook_update_N or other means.
    • No updates are necessary for this change.
  • Front end fixes should be tested against all of the Open Y Themes.
    • Tested against Carnation
    • Tested against Lily
    • Tested against Rose
    • This change does not contain front-end fixes.
  • I have flagged this PR "Needs Review" or pinged the VY devs/QA
    team in Slack

@fivejars fivejars deleted a comment from fjbot Feb 26, 2021
@fivejars fivejars deleted a comment from fjbot Feb 26, 2021
@fivejars fivejars deleted a comment from fjbot Feb 26, 2021
@fivejars fivejars deleted a comment from froboy Feb 28, 2021
@hamrant hamrant merged commit 80e2416 into master Feb 28, 2021
@hamrant hamrant deleted the fix_recurring_events_version branch October 11, 2021 15:07
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.

3 participants