Skip to content
This repository was archived by the owner on Oct 4, 2023. It is now read-only.

Revert "Remove tik tok feature flag"#1307

Merged
sliptype merged 1 commit into
mainfrom
sk-472-add-back-tiktok-flag
May 11, 2022
Merged

Revert "Remove tik tok feature flag"#1307
sliptype merged 1 commit into
mainfrom
sk-472-add-back-tiktok-flag

Conversation

@sliptype

Copy link
Copy Markdown
Contributor

This reverts commit d3df9a8.

Description

Dragons

Is there anything the reviewer should be on the lookout for? Are there any dangerous changes?

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide repro instructions & any configuration.

How will this change be monitored?

For features that are critical or could fail silently please describe the monitoring/alerting being added.

@dylanjeffers dylanjeffers left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice, so are there cases where we have tiktok stuff still around that isn't actually relying on the feature? like i know share modals maybe?

@sliptype

Copy link
Copy Markdown
Contributor Author

Nice, so are there cases where we have tiktok stuff still around that isn't actually relying on the feature? like i know share modals maybe?

What do you mean by 'relying on the feature'? The feature flag? All the UI is gated behind the flag afaik

@sliptype sliptype merged commit 6a0a7c5 into main May 11, 2022
@sliptype sliptype deleted the sk-472-add-back-tiktok-flag branch May 11, 2022 19:54
@sliptype sliptype mentioned this pull request May 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants