-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor: 🎨 migrated account component to TSX #47
refactor: 🎨 migrated account component to TSX #47
Conversation
⏳ Generating Lighthouse report... |
packages/account/src/Containers/Account/__tests__/account.spec.tsx
Outdated
Show resolved
Hide resolved
packages/account/src/Containers/Account/__tests__/account.spec.tsx
Outdated
Show resolved
Hide resolved
packages/account/src/Containers/Account/page-overlay-wrapper.tsx
Outdated
Show resolved
Hide resolved
@@ -9,7 +9,7 @@ import ProofOfOwnership from 'Sections/Verification/ProofOfOwnership'; | |||
import TwoFactorAuthentication from 'Sections/Security/TwoFactorAuthentication'; |
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.
kindly migrate index to ts 🚚
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.
I am getting Parse errors in imported module 'Sections/Verification/ProofOfAddress': ';' expected.
I am guessing it wants the imported and exported to be tsx?
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.
in the respective index file of ProofOfAdress
, can you write
export { default } from ****
instead of
export default from ****
will this work?
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.
I think on migrating the imported files to tsx, this error should go. So I'll make those changes seperately?
cd01ce8
into
sprint-9/account-package-refactor
…om#9832) * refactor: ♻️ initial commit * refactor: 🎨 fixed import file paths * fix: 🧑💻 incorporate sonarcloud review * refactor: ♻️ migrated reset-trading-password component to TSX * refactor: 🎨 resolved build failure * fix: ♻️ incorporated lint errors * fix: 🎨 incorporated review comments * fix: 🐛 resize issue * refactor: 🎨 migrated account component to TSX (#47) * refactor: 🎨 migrated account component to TSX * fix: 🎨 refactored code * refactor: 🎨 migrated account to tsx * fix: 🎨 incorporated review comments * chore: 💚 trigger empty commit * fix: 🧪 failing testcase * fix: 🐛 resolve password modal API failure * fix: 🐛 bug regd disabled Financial Assessment * fix: 🚚 updated paths * Merge branch 'master' into sprint-9/account-package-refactor
Changes:
Please provide a summary of the change.
Screenshots:
Please provide some screenshots of the change.