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: removed enable_moderation_reason_codes flag #615

Merged
merged 2 commits into from
Dec 3, 2023

Conversation

AhtishamShahid
Copy link
Contributor

@AhtishamShahid AhtishamShahid commented Dec 1, 2023

description.

Deprecated enable_moderation_reason_codes flag from the discussion app

Ticket

https://2u-internal.atlassian.net/browse/INF-1159

fix: resolved linter error

fix: changed workflow

fix: changed workflow

fix: changed workflow

fix: changed workflow

fix: changed workflow

fix: changed workflow

fix: changed workflow

fix: changed workflow

fix: changed workflow

fix: changed workflow

fix: changed workflow

fix: changed workflow
sundasnoreen12
sundasnoreen12 previously approved these changes Dec 1, 2023
Copy link
Contributor

@sundasnoreen12 sundasnoreen12 left a comment

Choose a reason for hiding this comment

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

PR LGTM. @AhtishamShahid please check test cases are failing.

@sundasnoreen12 sundasnoreen12 dismissed their stale review December 1, 2023 10:44

I'll review again once the failed test cases will pass.

@awais-ansari awais-ansari changed the title chore: removed deprecated flag feat: removed deprecated flag Dec 1, 2023
@awais-ansari awais-ansari changed the title feat: removed deprecated flag feat: removed enable_moderation_reason_codes flag Dec 1, 2023
Copy link

codecov bot commented Dec 1, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a479f5a) 92.37% compared to head (10cdb92) 92.36%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #615      +/-   ##
==========================================
- Coverage   92.37%   92.36%   -0.02%     
==========================================
  Files         169      169              
  Lines        3450     3445       -5     
  Branches      898      897       -1     
==========================================
- Hits         3187     3182       -5     
  Misses        243      243              
  Partials       20       20              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@sundasnoreen12 sundasnoreen12 left a comment

Choose a reason for hiding this comment

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

PR LGTM

@sundasnoreen12 sundasnoreen12 merged commit c6d953f into master Dec 3, 2023
7 checks passed
@sundasnoreen12 sundasnoreen12 deleted the ahtisham/INF-1159 branch December 3, 2023 17:49
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