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

Dashboard Integration #120

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

Conversation

dev-blc
Copy link
Member

@dev-blc dev-blc commented Jul 16, 2024

COPY OF PR BY @Immadominion #116

Title

Integrated BE logic with TS.

Description

Consumed the Backend with TS and displayed in the UI on the home screen on the mobile dashboard. I just followed the processes as seen on the desktop, nothing new.

Changes Made

  • Consumed endpoints for address
  • Consumed endpoints for total user balance
  • Consumed endpoints for user assets balance
  • Consumed endpoints for user nfts balance
  • Consumed endpoints for user activity history
  • Handled empty case for user assets
  • Handled empty case for nfts
  • Handled empty case for actvity history
  • Commented out the user profit or loss indicator directly below the total balance (The one that show in green or red and is in percentages. As this feature has not been implemented on the backend)

Checklist

  • Code compiles without errors.
  • Tested only on simulator, not with code.
  • making PR as requested by team lead.

Additional Notes

  • Added images from the figma design (Edited the images by merging them on the figma software, then imported them in the code)

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.

1 participant