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

Update data.ts to sync panels #60

Closed
Codehagen opened this issue Jan 16, 2024 · 11 comments · Fixed by #72
Closed

Update data.ts to sync panels #60

Codehagen opened this issue Jan 16, 2024 · 11 comments · Fixed by #72
Assignees
Milestone

Comments

@Codehagen
Copy link
Member

Codehagen commented Jan 16, 2024

Currently, our interface shows hardcoded data on the right panel when a user selects a stock or an account. We need to modify this behavior to dynamically update these fields with actual stock data retrieved from data.ts, replacing the hardcoded content.

This you can see working on the Accounts page

Screenshot:
jam-screenshot

Full screen:
jam-full-url-screenshot

-------------------------------------

Website URL:
http://localhost:3000/dashboard/investments

Device and browser info:
Chrome 120.0.6099.216 (1440x788) | Mac OS (arm) 10.15.7

Date and time:
January 16th 2024 | 8:07pm UTC

View developer information (console logs, network requests & timing):
jam.dev/c/c2e2e33a-2752-4cad-b1ad-252c54a3f670

@Codehagen Codehagen added the ui label Jan 16, 2024
@miralsuthar
Copy link
Contributor

Can I work on this?

@Codehagen
Copy link
Member Author

Yes! I started something. We should try to have somewhat same layout as the prisma schema 👏🏻

@Codehagen Codehagen added this to the Ui Done milestone Jan 17, 2024
@Codehagen
Copy link
Member Author

If you have any trouble @miralsuthar just give me a ping and i would love to help ✨

@miralsuthar
Copy link
Contributor

Sure, If I get stuck somewhere I will let you know 👍

@miralsuthar
Copy link
Contributor

Hello, @meglerhagen

https://www.loom.com/share/aeebfb84a21846e6bc87f7f2631d20b3?sid=b82af339-5594-4ceb-bc98-6f054915ee34

Is this correct. I think the data in data.tsx file is bit upside down 😅.

@Codehagen
Copy link
Member Author

Going to look at it 👍

@Codehagen
Copy link
Member Author

This was very good @miralsuthar! Would love to have you look at the rest of the component so we can click and update the right side ✨

@miralsuthar
Copy link
Contributor

Thanks,

I would love to work on the rest of the component @meglerhagen. Is there any issue open for this? or I can directly work on this?

@Codehagen
Copy link
Member Author

No new issue, you want to make or should i make it? :) Thank you!

@miralsuthar
Copy link
Contributor

It would be great if you create an issue for it, will be more helpful for me to understanding the requirements.

Thank you.

@Codehagen
Copy link
Member Author

Going to do it in a few hours. Looks good so far.
Also going to look at the backend so the stichting is going got fast for what we choose ✨

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 a pull request may close this issue.

2 participants