goinvo / staffplan-next-app

Next App for Staffplan
https://staffplan-next-app.vercel.app
1 stars 1 forks source link

Incorporated context and bar rendering #47

Closed MyStarrySpace closed 6 months ago

MyStarrySpace commented 6 months ago

Added contexts to pass variables between pages easily, reducing redundant backend requests and need for cache. Also added basic rendering of bars (there are still some issues, such as:

vercel[bot] commented 6 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
staffplan-next-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 25, 2024 6:07pm
HankC138 commented 6 months ago

I'm sure we'll need to deal with a fair amount of conflicts but that's understandable with such a large rework. Thank you for this big refactor 🙏 . Should we add in my ellipsis menu functionality before or after the change as it exists based on the previous arrangment of files? No biggie either way.

Would you like to share your linter/prettier settings so we can can sync that up as well so we have less formatting changes in our commits?

MyStarrySpace commented 6 months ago

I'm sure we'll need to deal with a fair amount of conflicts but that's understandable with such a large rework. Thank you for this big refactor 🙏 . Should we add in my ellipsis menu functionality before or after the change as it exists based on the previous arrangment of files? No biggie either way.

I suppose it makes more sense to update the bigger context-related changes first, since so much of the project structure was changed, and the ellipsis menus mainly affect the sidebar and functions passed into it, which aren't a super huge change

Would you like to share your linter/prettier settings so we can can sync that up as well so we have less formatting changes in our commits?

I'm using Ctrl+Shift+F in vscode to auto-format. Let me look up how to export those settings

MyStarrySpace commented 6 months ago

Correction: Shift+Alt+F, not Ctrl-Shift

HankC138 commented 6 months ago

Good to go then feel free to resolve any conflicts and merge this one! Thanks again for all this! 🚀