-
Notifications
You must be signed in to change notification settings - Fork 96
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
refactor (Components) : Refactored Components #179
base: main
Are you sure you want to change the base?
Conversation
Hi @Raunakk02 , can you please squash your commits and follow the commit message guidelines. |
…ble', and 'UploadReuse' components
Hi @sjha2048, thanks for reviewing the PR. As suggested by you, I have squashed and updated the commits as per the commit message guidelines. Please let me know if any other change is needed. |
When I login, I see following error:
|
@GMishx please have a look at this PR created by @krishna9304. He has fixed the issue. It was out of the scope of this particular PR and some other contributors (including me) was facing it too. |
This pull request has conflicts, please rebase to resolve those before we can evaluate the pull request. |
Description
Refactored the files inside
src/components
folderChanges
Implemented the changes as proposed in this comment of issue #108
How to test
Since the changes are just related to refactoring, simple testing of the UI components should be sufficient.
fixes #108