Skip to content

Conversation

tiina303
Copy link
Contributor

@tiina303 tiina303 commented Feb 28, 2023

Problem

On top of #14445
Making the migration a no-op

will merge this into the below one, but for easier review created separately

Changes

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

How did you test this code?

@posthog-bot
Copy link
Contributor

Hey @tiina303! 👋
This pull request seems to contain no description. Please add useful context, rationale, and/or any other information that will help make sense of this change now and in the distant Mars-based future.

@tiina303 tiina303 changed the title make 0305 a no-op fix: make 0305 a no-op migration Feb 28, 2023
@tiina303 tiina303 requested a review from timgl February 28, 2023 14:44
@timgl
Copy link
Collaborator

timgl commented Feb 28, 2023

Let's get this in before #14445

@timgl timgl marked this pull request as ready for review February 28, 2023 14:46
@tiina303 tiina303 force-pushed the no-op-the-migration branch from 6f147e5 to f7aebdf Compare February 28, 2023 14:51
@tiina303 tiina303 merged commit 4e8c4ec into revert-14277-fix/add-person-consrtaint-on-override Feb 28, 2023
@tiina303 tiina303 deleted the no-op-the-migration branch February 28, 2023 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants