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

feat: Settings 3000 #18419

Merged
merged 65 commits into from
Nov 9, 2023
Merged

feat: Settings 3000 #18419

merged 65 commits into from
Nov 9, 2023

Conversation

benjackwhite
Copy link
Contributor

@benjackwhite benjackwhite commented Nov 6, 2023

Problem

Inspired by this lets make settings way better.

  1. Make it properly sectioned
  2. Make it portable so we can embed product specific settings where they need to be
  3. Clean up the components
  4. Put it in the side panel
  5. Remove as much antd as possible

Changes

Screenshot 2023-11-06 at 17 10 27 Screenshot 2023-11-07 at 11 20 03

TODO

  • Make sure it works with mobile
  • Fix hash params updating
  • Fix it for non-3000
  • Make SessionReplay modal use it

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

How did you test this code?

@benjackwhite benjackwhite marked this pull request as ready for review November 6, 2023 16:11
@benjackwhite benjackwhite requested review from a team and corywatilo November 7, 2023 10:19
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

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

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

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

  • chromium: 0 added, 3 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.

benjackwhite and others added 3 commits November 8, 2023 12:59
# Conflicts:
#	frontend/src/layout/navigation/navigationLogic.ts
#	frontend/src/lib/components/ActivationSidebar/activationLogic.test.ts
#	frontend/src/lib/components/ActivationSidebar/activationLogic.ts
#	frontend/src/lib/components/DefinitionPopover/DefinitionPopover.tsx
#	frontend/src/lib/components/Subscriptions/views/EditSubscription.tsx
#	frontend/src/scenes/App.tsx
#	frontend/src/scenes/dashboard/dashboards/DashboardsTable.tsx
#	frontend/src/scenes/notebooks/NotebooksTable/NotebooksTable.tsx
#	frontend/src/scenes/organization/Settings/membersLogic.tsx
#	frontend/src/scenes/organization/membersLogic.tsx
#	frontend/src/scenes/saved-insights/SavedInsightsFilters.tsx
#	frontend/src/scenes/session-recordings/saved-playlists/SavedSessionRecordingPlaylists.tsx
#	frontend/src/scenes/settings/organization/membersLogic.tsx
#	frontend/src/scenes/settings/project/teamMembersLogic.tsx
#	frontend/src/scenes/settings/user/TwoFactorAuthentication.tsx
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

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

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

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

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

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

Triggered by this commit.

👉 Review this PR's diff of snapshots.

github-actions bot and others added 2 commits November 8, 2023 15:17
# Conflicts:
#	frontend/__snapshots__/posthog-3000-navigation--navigation-3000.png
#	frontend/__snapshots__/posthog-3000-navigation--navigation-base.png
#	frontend/__snapshots__/scenes-app-notebooks--recordings-playlist.png
#	frontend/src/scenes/appScenes.ts
#	frontend/src/scenes/sceneLogic.test.tsx
#	frontend/src/scenes/sceneTypes.ts
#	frontend/src/scenes/scenes.ts
@posthog-bot
Copy link
Contributor

📸 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.

benjackwhite and others added 6 commits November 9, 2023 08:58
# Conflicts:
#	frontend/src/layout/navigation/TopBar/TopBar.tsx
#	frontend/src/scenes/project/Settings/AccessControl.tsx
#	frontend/src/scenes/settings/organization/membersLogic.tsx
#	frontend/src/scenes/settings/project/ProjectAccessControl.tsx
#	frontend/src/scenes/settings/user/PersonalAPIKeys.tsx
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

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

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@benjackwhite benjackwhite merged commit 409b723 into master Nov 9, 2023
73 checks passed
@benjackwhite benjackwhite deleted the feat/settings-3000 branch November 9, 2023 09:08
@corywatilo corywatilo mentioned this pull request Nov 27, 2024
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