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

fix(experiments): JS snippet typo #17434

Merged
merged 6 commits into from
Sep 14, 2023
Merged

Conversation

Lior539
Copy link
Contributor

@Lior539 Lior539 commented Sep 14, 2023

Problem

Small typo for on experiments page JS code to override feature flags. Snippet used snake_case instead of camelCase

Reported by user here -> https://posthogusers.slack.com/archives/CTLTM70RM/p1694643924884199

How did you test this code?

👀

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

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

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.

@neilkakkar neilkakkar changed the title fix experiments JS snippet typo fix(experiments): JS snippet typo Sep 14, 2023
Copy link
Collaborator

@neilkakkar neilkakkar left a comment

Choose a reason for hiding this comment

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

lovely thank you!

@neilkakkar neilkakkar enabled auto-merge (squash) September 14, 2023 08:59
@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 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@benjackwhite benjackwhite removed their request for review September 14, 2023 09:06
@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 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@neilkakkar neilkakkar merged commit 49fc588 into master Sep 14, 2023
@neilkakkar neilkakkar deleted the fix-experiments-code-snippet-typo branch September 14, 2023 09:35
pauldambra pushed a commit that referenced this pull request Sep 14, 2023
daibhin added a commit that referenced this pull request Sep 14, 2023
pauldambra added a commit that referenced this pull request Sep 14, 2023
* fix: universal search with notebooks disabled

* Update frontend/src/lib/components/TaxonomicFilter/taxonomicFilterLogic.tsx

Co-authored-by: Paul D'Ambra <[email protected]>

* Update UI snapshots for `chromium` (1)

* fix(experiments): JS snippet typo (#17434)

* chore: clear the console in storybook a little (#17430)

* chore: clear the console in storybook a little

* Update .storybook/preview.tsx

* Update UI snapshots for `chromium` (1)

* Update UI snapshots for `chromium` (1)

* maybe this?

* Update UI snapshots for `chromium` (1)

* Update UI snapshots for `chromium` (1)

* maybe this

* Update UI snapshots for `chromium` (1)

* Update UI snapshots for `chromium` (1)

* Update UI snapshots for `chromium` (2)

* Update UI snapshots for `chromium` (2)

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: Paul D'Ambra <[email protected]>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
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