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.

Sign up for GitHub

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

Jump to bottom

Refactor front #787

Closed
wants to merge 13 commits into from
Closed

Refactor front #787

wants to merge 13 commits into from

Conversation

DimaAmega
Copy link
Collaborator

@DimaAmega DimaAmega commented Oct 1, 2023 •
edited
Loading

  • add new version of projects/:projectId/:page?/:reportId? route with loaders to get smooth UI
  • create switch to old version for better testing
  • Make it all just work
  • restructure Projects2 folder to use ... operator to inject element, loader into router
  • add global breadcrumbs
  • build assets and move it to ui folder, make commit with it
    New checks:
  • Refactor code: create Routes folder

@DimaAmega DimaAmega mentioned this pull request Oct 1, 2023
Closed
@DimaAmega DimaAmega linked an issue Oct 1, 2023 that may be closed by this pull request
UI Refactor Navigation #786
Closed
@DimaAmega DimaAmega closed this Oct 2, 2023
@DimaAmega DimaAmega removed a link to an issue Oct 2, 2023
UI Refactor Navigation #786
Closed
@mike0sv mike0sv deleted the refactor-front branch April 9, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@DimaAmega