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

[FE] Integrate Portfolio Management with Backend #394

Closed
5 tasks done
hikasap opened this issue Nov 15, 2024 · 2 comments · Fixed by #475
Closed
5 tasks done

[FE] Integrate Portfolio Management with Backend #394

hikasap opened this issue Nov 15, 2024 · 2 comments · Fixed by #475
Assignees
Labels
frontend issue related with frontend priority: medium Urgent issue status: in review Issue is under review workload: 3 issues that are requiring high effort

Comments

@hikasap
Copy link
Contributor

hikasap commented Nov 15, 2024

Description

This task focuses on integrating the Portfolio Management feature with backend services to enable persistent storage and retrieval of user portfolios. Currently, portfolios can be created using modals and mock data. With this integration:

  • Portfolios will be stored in the backend using API calls.
  • Users will be able to see their portfolios on the screen by fetching portfolio data from the backend instead of relying on mock data.

The task involves updating the portfolio creation flow to send data to the backend and modifying the portfolio display components to dynamically populate information retrieved via API calls.

Addressed Requirements:

  • 1.2.2.1 Portfolios should be created by the users by entering purchase price and quantity of stock.
  • 1.2.2.2 Portfolios shall include one or more stock.

Required Activities

  • Implement API calls to store portfolios created by users in the backend.
  • Replace mock data in the portfolio display with data fetched from the backend.
  • Ensure that portfolio details are dynamically displayed on the user’s screen.
  • Add error handling for failed API calls and loading states for a better user experience.
  • Test the integration for smooth operation and resolve any potential bugs.

Estimated Time

12 hours

Deadline

25 Nov 2024

Reviewer

@mahmutbugramert

@hikasap hikasap added priority: medium Urgent issue workload: 3 issues that are requiring high effort frontend issue related with frontend labels Nov 15, 2024
@hikasap hikasap added this to the 451 - Milestone 2 milestone Nov 15, 2024
@hikasap hikasap self-assigned this Dec 12, 2024
@hikasap hikasap linked a pull request Dec 12, 2024 that will close this issue
@hikasap
Copy link
Contributor Author

hikasap commented Dec 12, 2024

I have created a PR related to this. I am kindy asking for your rev @mahmutbugramert. Once the pr is merged I think we can close this issue.

@hikasap hikasap added the status: in review Issue is under review label Dec 12, 2024
@mahmutbugramert
Copy link
Contributor

Since the PR is merged I am closng the issue. Thanks'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend issue related with frontend priority: medium Urgent issue status: in review Issue is under review workload: 3 issues that are requiring high effort
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants