Skip to content

Removing update table user plan#4551

Merged
zomars merged 4 commits into
mainfrom
fix/reverting-plan-conversion-migration
Sep 17, 2022
Merged

Removing update table user plan#4551
zomars merged 4 commits into
mainfrom
fix/reverting-plan-conversion-migration

Conversation

@leog
Copy link
Copy Markdown
Contributor

@leog leog commented Sep 17, 2022

What does this PR do?

This is to avoid having the wrong conversion being applied each deployment now that we figured that we don't need every user to be with a FREE plan.

Environment: Staging(main branch) / Production

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@leog leog requested a review from a team September 17, 2022 14:38
@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 17, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal ✅ Ready (Inspect) Visit Preview Sep 17, 2022 at 8:28PM (UTC)

Copy link
Copy Markdown
Contributor

@alannnc alannnc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One line change LGTM

Copy link
Copy Markdown
Contributor

@zomars zomars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks simple enough but I'd rather create a new migration. Blocking until coming up with a fix.

Copy link
Copy Markdown
Contributor

@zomars zomars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@zomars zomars enabled auto-merge (squash) September 17, 2022 20:21
@zomars zomars merged commit e7dd0ff into main Sep 17, 2022
@zomars zomars deleted the fix/reverting-plan-conversion-migration branch September 17, 2022 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants