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

(dev/core#2122) CiviEvent - Add pre-upgrade warning #22958

Merged
merged 3 commits into from
Mar 17, 2022

Conversation

totten
Copy link
Member

@totten totten commented Mar 17, 2022

Overview

Add pre-upgrade warning about the current CiviEvent/timezone issues.

cc @agh1 @KarinG @demeritcowboy @seamuslee001

https://lab.civicrm.org/dev/core/-/issues/2122

After

Screen Shot 2022-03-16 at 5 55 50 PM

@civibot
Copy link

civibot bot commented Mar 17, 2022

(Standard links)

@civibot civibot bot added the 5.47 label Mar 17, 2022
@agh1
Copy link
Contributor

agh1 commented Mar 17, 2022 via email

@demeritcowboy
Copy link
Contributor

I haven't r-run'd it but looks good.

And oh cool there's even a status page devoted to this issue on civicrm.org now.

@totten
Copy link
Member Author

totten commented Mar 17, 2022

Merging. It's got 👍 from @agh1 and @demeritcowboy, and it's a bit time-sensitive. CI hasn't finished yet, but the upgrade-test ran OK; it's unlikely to have relevant coverage anywhere else; and I did do some r-run on both laptop+desktop.

@totten totten merged commit d7f26ca into civicrm:5.47 Mar 17, 2022
@totten totten deleted the 5.47-event-tz-warn branch March 17, 2022 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants