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

Revert "Increase Element model varchar limits" #3923

Merged
merged 1 commit into from
Apr 8, 2021

Conversation

Twixes
Copy link
Member

@Twixes Twixes commented Apr 8, 2021

Reverts #3912 because we're seeing some yet unexplained issue that prevents this from migrating.

@Twixes Twixes added automerge Merge this PR automatically when REQUIRED checks pass and removed automerge Merge this PR automatically when REQUIRED checks pass labels Apr 8, 2021
@timgl timgl temporarily deployed to posthog-pr-3923 April 8, 2021 19:22 Inactive
@posthog-bot posthog-bot merged commit e30681b into master Apr 8, 2021
@posthog-bot posthog-bot deleted the revert-3912-increase-element-varchars branch April 8, 2021 19:24
Twixes added a commit that referenced this pull request Apr 8, 2021
fuziontech pushed a commit that referenced this pull request Apr 19, 2021
…o 3765-cohort-by-trend

* '3765-cohort-by-trend' of github.com:PostHog/posthog: (39 commits)
  'string, parsable as datetime' (#3942)
  Update plugin server to 0.16.3 (#3944)
  Resizable table columns in Sessions (#3927)
  bump cryptography==3.4.7 and add macosx_arm64 install script (#3935)
  🤖: Add jeduden as a contributor 🎉 (#3938)
  Fix feature flags default rollout (#3745)
  Less dancing in dashboards (#3824)
  Always show event stats and add warnings (#3908)
  Update plugin server to 0.16.2 (#3932)
  Minimum PostHog version in plugins (#3916)
  Renames Active users to Unique users (#3930)
  Fix action with same name (#3909)
  Fix navigation to insights from dashboards (#3928)
  User V2 Part II - Frontend changes (#3866)
  update autocapture label to be more descriptive (#3925)
  Log to sentry when migrations are out of date (#3924)
  Revert "Increase Element model varchar limits (#3912)" (#3923)
  Update plugin server to 0.16.1 (#3920)
  Are migrations safe to run on cloud?  (#3917)
  Run Automerge as posthog-bot (#3919)
  ...
Twixes added a commit that referenced this pull request May 6, 2021
* Revert "Revert "Increase Element model varchar limits (#3912)" (#3923)"

This reverts commit e30681b.

* Use RunSQL to avoid ORM crap

* Fix attr_class being converted from varchar[] to varchar

* Bump migration

* Simplify migration and remove attr_class alter due to locking

* Bump migration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge this PR automatically when REQUIRED checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants