-
Notifications
You must be signed in to change notification settings - Fork 2
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
#187419123-Feature Users should be able to Update their Password #12
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
UpdatePasswordPayload, | ||
{ rejectValue: UpdatePasswordError } | ||
>("updatePassword", async (payload, { rejectWithValue }) => { | ||
try { |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
initialState, | ||
reducers: {}, | ||
extraReducers: (builder) => { | ||
builder |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #12 +/- ##
==========================================
+ Coverage 64.88% 69.57% +4.69%
==========================================
Files 13 20 +7
Lines 131 309 +178
Branches 5 25 +20
==========================================
+ Hits 85 215 +130
- Misses 46 94 +48 ☔ View full report in Codecov by Sentry. |
e62e69d
to
864885d
Compare
864885d
to
bb05637
Compare
bb05637
to
f3069fe
Compare
|
GitGuardian id | GitGuardian status | Secret | Commit | Filename | |
---|---|---|---|---|---|
12081800 | Triggered | Generic Password | 725aa7a | src/test/updatePasswordApiSlice.test.tsx | View secret |
12081800 | Triggered | Generic Password | 725aa7a | src/test/updatePasswordApiSlice.test.tsx | View secret |
12081800 | Triggered | Generic Password | 725aa7a | src/test/updatePasswordApiSlice.test.tsx | View secret |
12081800 | Triggered | Generic Password | 725aa7a | src/test/updatePasswordApiSlice.test.tsx | View secret |
🛠 Guidelines to remediate hardcoded secrets
- Understand the implications of revoking this secret by investigating where it is used in your code.
- Replace and store your secrets safely. Learn here the best practices.
- Revoke and rotate these secrets.
- If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
- following these best practices for managing and storing secrets including API keys and other credentials
- install secret detection on pre-commit to catch secret before it leaves your machine and ease remediation.
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
src/__test__/updatePassword.test.tsx
Outdated
import UpdatePasswordmod from "../components/password/UpdateModal"; | ||
import store from "../redux/store"; | ||
|
||
test("test update password", () => { |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
src/__test__/updatePassword.test.tsx
Outdated
userEvent.click(updateButton); | ||
}); | ||
|
||
test("test update password with mismatching new password and confirm password", () => { |
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.
Similar blocks of code found in 2 locations. Consider refactoring.
f3069fe
to
a1de45f
Compare
a1de45f
to
83992b0
Compare
83992b0
to
ff03571
Compare
ff03571
to
fa917cc
Compare
fa917cc
to
aa84faf
Compare
aa84faf
to
6422830
Compare
6422830
to
bd66387
Compare
#187419122 Seller Authentication and Two factor authentication for seller
bd66387
to
725aa7a
Compare
Code Climate has analyzed commit 725aa7a and detected 4 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
what Does this PR do?
How to test the functionality ?