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

Portfolio Screen & Asset Details #526

Merged
merged 3 commits into from
Dec 15, 2024

Conversation

Pqrq
Copy link
Contributor

@Pqrq Pqrq commented Dec 15, 2024

PR Description: Fix Portfolio Screen and Enhance Asset Details Page

Summary

This PR addresses key fixes and enhancements to the Portfolio Screen and introduces improvements to the Asset Details Page for a more seamless and interactive user experience:

  1. Portfolio Screen Fixes:

    • Resolved navigation issues and ensured proper routing between screens.
    • Improved the display of portfolio data, including asset details and charts.
    • Enhanced UI components for better responsiveness and usability.
  2. Asset Details Page Enhancements:

    • Improved the layout and styling of asset information, charts, and news sections.
    • Ensured proper rendering of dynamic content based on selected assets.
  3. Code Improvements:

    • Refactored reusable components and streamlined state management for easier maintainability.
    • Enhanced error handling and optimized API calls for asset data retrieval.
    • Reduced redundancy by consolidating shared styles and functions.

Changes

  • Fixed navigation behavior in the Portfolio page.
  • Updated the Portfolio Screen to display asset data dynamically and improve the user interface.
  • Benefited from reusable components for charts, date ranges, and news sections to reduce code duplication.
  • Refined state management and error handling across the portfolio and asset pages.
  • Made minor style adjustments for better consistency across the app.

Did some fixes at the Portfolio Screen
@Pqrq Pqrq added Enhancement New feature or request Priority: High Requires taking action ASAP Frontend Mobile mobile app development labels Dec 15, 2024
Copy link
Contributor

@karatashsyn karatashsyn left a comment

Choose a reason for hiding this comment

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

Looks perfect !!

@Pqrq Pqrq merged commit 8036eff into mobile-main-updated Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request Frontend Mobile mobile app development Priority: High Requires taking action ASAP
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants