search

Rafael4DC / AssemblyOdin

Assembly point management system - Codename: Odin
Other
2 stars 0 forks source link

Feature/#68 frontend wireframe #69

Closed Rafael4DC closed 7 months ago

Rafael4DC commented 7 months ago

Hi! I want to check if the logic used, especially on the hooks folder, and the API calls on the services section make sense. You can fully ignore apiFetch, as its still 100% WIP The rest i want to make double sure everything makes sense, as it should, from what I read

Rafael4DC commented 7 months ago

@YuriValle

Rafael4DC commented 7 months ago

Added another commit, this time more relevant Besides what's written in the commit message its important to keep in mind I decided to use react bootstrap because its closer to how jetpack compose works, which both developers know how to use better But regular bootstrap can still be used, wanted to mainly demo it to look for approval in its usage, from both relevant parties @bfmserra @YuriValle

YuriValle commented 7 months ago

It still needs work, but it's much better now. Checkboxes don't work as expected.

Please avoid creating things with 'Work In Progress'. If you don't have it now, you don't put it, specially because it's NOT in progress, you're doing something else.

YuriValle commented 7 months ago

The concept of DTO does not exist in a frontend context. This task is being developed way too far from what the scope of it was meant to be.

YuriValle commented 7 months ago

Whenever you're ready to stop with this task and move on to another task. Feel free.