Skip to content

Conversation

@vitormattos
Copy link
Member

⚠️
ATTENTION!!

This PR was created from the branch of #2037

Before review this PR, is necessary merge #2037 to reduce the quantity of changes.

Motivations of this PR

  • Organize the frontend routes to follow the same pattern of other apps
  • Fixes that was identified during manual tests
  • Move part of logic from frontend to backend

@vitormattos vitormattos changed the title Feature/refactor router Refactor frontend router Dec 31, 2023
@vitormattos vitormattos self-assigned this Dec 31, 2023
@vitormattos vitormattos added this to the Next Major (29) milestone Dec 31, 2023
Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
I made a grep by ExternalRoute and have not a reference to this name

Signed-off-by: Vitor Mattos <[email protected]>
Reduce dependencies

Signed-off-by: Vitor Mattos <[email protected]>
* Moved message to backend to remove require the translation pack at routes
* Fix usage of initial state
* Remove unecessary prop because was moved to initial state

Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
These endpoints may return errors if is not an authenticated user and
this is not a problem, only signalize that the user have not a passowrd
to sign and have not a visible signature.

Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
Is no more necessary to maintain outside

Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
@vitormattos vitormattos force-pushed the feature/refactor-router branch from 4304c74 to 11224d0 Compare January 2, 2024 13:14
Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
Signed-off-by: Vitor Mattos <[email protected]>
Now was need to mock a more property

Signed-off-by: Vitor Mattos <[email protected]>
@vitormattos vitormattos marked this pull request as ready for review January 2, 2024 14:52
@vitormattos
Copy link
Member Author

/backport to stable28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants