Discussion000 - File structure

[alonso284]

There's clearly NO structure for all existing pages. trpc methods, react hooks, utils and variables are all over the place and unorganized. We should come up with a standard and stick to it.

Suggestions:

Some pages like src/app/(logged)/project/[projectId]/scrumboard/ItemsKanban.tsx already have some type of organization.

• GENERAL
• TRPC
• REACT
• UTILITY
  I'm not completely sure what each contains. We should write some rules on what they specifically each mean.

Also, we should use the #region syntax.

@SergioGzzBrz has been the pioneer of this standard. I would love to hear your input.

[SergioGzzBrz]

I think these should be the bare minimum (AT LEAST first 2) . They are generally used in all complex components, and usually have no problems being in that order (REACT being the most likely cause for any conflicts, and probably the least defined).

Utility can be broken into more regions if needed, since some views have special needs, and REACT can be straight up ignored if it does not fit the view well, but the idea stands.

We could probably ignore these for very small components, but for main views these are a must have

[alonso284]

I've been exploring different files and think the next structure can work

• React Hooks
  • useParam
  • useState
  • useEffect
  • useMemo
  • other
• Wrapped Hooks (everything inside ~/app/_hooks/)
  • Specialized hooks (itemOperationHooks, requirementHook, etc)
  • Invalidates
• TRPC
  • api.utils()
  • useQuery
  • useMutation
• consts and lets (variable, functions and dynamic variables [like filters and maps])

am I missing something?
of course this is just a big nit pick and we should only get to it if we have time to clean up the repo

[SergioGzzBrz]

I think that the first group "React Hooks" should be divided into 2 sections. One before TRPC and one after TRPC.

I've seen a few times the useEffects use the variables from TRPC, so these hooks need to be after TRPC.

However, other hooks, like useState sometimes need to be put before TRPC and sometimes after TRPC.

And, other hooks like useParam, always can go before TRPC (and most of the time they are needed for the queries)