-
Notifications
You must be signed in to change notification settings - Fork 87
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 (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