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.

Sign up for GitHub

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

Jump to bottom

feat: added toggle for forum data dog #31

Merged
merged 1 commit into from
Jun 24, 2024
Merged

feat: added toggle for forum data dog #31

merged 1 commit into from
Jun 24, 2024

Conversation

AhtishamShahid
Copy link

@AhtishamShahid AhtishamShahid commented Jun 4, 2024 •
edited
Loading

Make sure that the following steps are done before merging:

  • Have a Site Reliability Engineer review the PR if you don't own all of the services impacted.
  • If you are adding any new default values that need to be overridden when this change goes live, update internal repos and add an entry to the top of the CHANGELOG.
  • Performed the appropriate testing.

@AhtishamShahid
feat: added toggle for forum data dog
de6f2df 
fix: move data dog config behind flag
@AhtishamShahid AhtishamShahid merged commit 24376a9 into master Jun 24, 2024
4 checks passed
@AhtishamShahid AhtishamShahid deleted the ahtisham/INF-1420 branch June 24, 2024 06:49
syedsajjadkazmii pushed a commit that referenced this pull request Jun 25, 2024
@AhtishamShahid @syedsajjadkazmii
Merge pull request #31 from edx/ahtisham/INF-1420
285feda 
feat: added toggle for forum data dog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@muhammadadeeltajamul muhammadadeeltajamul muhammadadeeltajamul approved these changes

@ayesha-waris ayesha-waris ayesha-waris approved these changes

@asadazam93 asadazam93 asadazam93 approved these changes

@saadyousafarbi saadyousafarbi Awaiting requested review from saadyousafarbi

@awais-ansari awais-ansari Awaiting requested review from awais-ansari

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AhtishamShahid @asadazam93 @ayesha-waris @muhammadadeeltajamul