Skip to content
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

Add new page add_deposit #410

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

GoSTEAN
Copy link

@GoSTEAN GoSTEAN commented Dec 19, 2024

Changes made:

  • added the add_deposit page

This closes #404

frontend/src/App.jsx Outdated Show resolved Hide resolved
frontend/src/pages/spotnet/dashboard/add_deposit.css Outdated Show resolved Hide resolved
frontend/src/components/TokenSelector/TokenSelector.jsx Outdated Show resolved Hide resolved
frontend/src/components/TokenSelector/TokenSelector.css Outdated Show resolved Hide resolved
frontend/src/pages/spotnet/dashboard/add_deposit.css Outdated Show resolved Hide resolved
Copy link
Collaborator

@whateverfw whateverfw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How is the progress with a tasks?
Also, please make sure you fixed all CI issues

frontend/src/components/TokenSelector.jsx Outdated Show resolved Hide resolved
@djeck1432
Copy link
Owner

@GoSTEAN Any updates?
all PR should be merged today, if you not finish your task you will be rewarded partially

@whateverfw
Copy link
Collaborator

@GoSTEAN sync with main and fix merge conflicts

@GoSTEAN
Copy link
Author

GoSTEAN commented Dec 21, 2024

Presently fixing the conflicts , Will be pushing soon

@GoSTEAN
Copy link
Author

GoSTEAN commented Dec 21, 2024

@whateverfw I have resolved the conflict on my local machine but it's still reflecting on Github. Pls what should i do ?

@whateverfw
Copy link
Collaborator

@GoSTEAN I'm not sure, did you resolve it correctly and pushed the changes to git?

@djeck1432
Copy link
Owner

@GoSTEAN To save time, open new open, but do NOT close this one

@GoSTEAN
Copy link
Author

GoSTEAN commented Dec 21, 2024

@djeck1432 I have created a new pull request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FRONTEND] Add new page add_deposit
3 participants