-
Notifications
You must be signed in to change notification settings - Fork 100
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: added country disabling feature #1084
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2u-main #1084 +/- ##
===========================================
+ Coverage 66.78% 67.12% +0.34%
===========================================
Files 51 51
Lines 849 864 +15
Branches 173 176 +3
===========================================
+ Hits 567 580 +13
- Misses 272 274 +2
Partials 10 10 ☔ View full report in Codecov by Sentry. |
src/profile/ProfilePage.test.jsx
Outdated
expect(tree).toMatchSnapshot(); | ||
}); | ||
|
||
it('test user with disabled country', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can use it.each since most of the code is the same, and we are only checking whether the country is disabled or not.
5f178b1
to
28c8daa
Compare
* feat: added country disabling feature * fix: lint errors * test: added test case for disabled countries * refactor: combined test cases
INF-1567
Description
disabledCountries
list will no longer be visible to users in the profile settings of other countries, and users will not be able to switch to disabled countries.For Example, if Russia is on the disabled countries list.
Note