posthog icon indicating copy to clipboard operation
posthog copied to clipboard

feat(environments): Add new environments endpoint and move /projects to Project

Open Twixes opened this issue 1 year ago â€ĸ 7 comments

Problem

Step 4 of project environments implementation.

Changes

Added the /environments endpoint, which is based on the Team model. At the same time, moved /projects to the Project model, with a backward compatibility feature of "passing through" reads and writes of environment-specific settings to the Team with the same ID as the Project (this means that all existing TestTeam tests should continue to pass).

TODO for self before merging: check if project.name == project.passthrough_team.name currently holds for all projects in both regions.

How did you test this code?

TODO: Add /environments tests.

Twixes avatar Aug 02 '24 08:08 Twixes

This PR hasn't seen activity in a week! Should it be merged, closed, or further worked on? If you want to keep it open, post a comment or remove the stale label – otherwise this will be closed in another week. If you want to permanentely keep it open, use the waiting label.

posthog-bot avatar Aug 12 '24 07:08 posthog-bot

This PR was closed due to lack of activity. Feel free to reopen if it's still relevant.

posthog-bot avatar Aug 19 '24 07:08 posthog-bot

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Aug 19 '24 16:08 posthog-bot

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

  • chromium: 0 added, 4 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Aug 19 '24 16:08 posthog-bot

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

  • chromium: 0 added, 4 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Aug 19 '24 17:08 posthog-bot

Size Change: 0 B

Total Size: 1.1 MB

â„šī¸ View Unchanged
Filename Size
frontend/dist/toolbar.js 1.1 MB

compressed-size-action

github-actions[bot] avatar Aug 21 '24 14:08 github-actions[bot]

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Aug 21 '24 14:08 posthog-bot

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Sep 09 '24 19:09 posthog-bot

Thanks for giving this a look @aspicer!

Twixes avatar Sep 10 '24 14:09 Twixes

📸 UI snapshots have been updated

1 snapshot changes in total. 0 added, 1 modified, 0 deleted:

  • chromium: 0 added, 1 modified, 0 deleted (diff for shard 2)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

posthog-bot avatar Sep 10 '24 19:09 posthog-bot