-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
FSD: shared folder refactor #345
Comments
Closed
8 tasks
8 tasks
8 tasks
Closed
Closed
Closed
This was referenced Jul 7, 2024
8 tasks
8 tasks
Closed
8 tasks
This was referenced Aug 14, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
📝 Describe the refactoring task
Shared folder refactor
📍 Code location
"@/shared/"
🛠️ Suggested improvement
"Example" PR
Презентация по FSD
✅ Definition of Done (DoD):
Everything on the list has been refactored
TO-DO
text-with-link - should be deleted- decided to perform separately as its own issue2) Data (related:- Data will be refactored separately as it's own milestone3) Assets + icons- Assets decided to leave as it is, icons can be refactored separately but not yet necessary4) hooks: use-course-by-title, use-data-by-name, app/services/api- Part of the huge milestone for data refactor (data will be fetched from external CDN so this hooks will be deleted)update 26.08.24:
"3. Assets + icons"
and"4. hooks: use-course-by-title, use-data-by-name, app/services/api"
- decided to exclude from FSD refactor and expected to perform as a separate issues after we fully complete the migrationThe text was updated successfully, but these errors were encountered: