Skip to content

Conversation

@seanaye
Copy link
Contributor

@seanaye seanaye commented Dec 3, 2025

Summary

This is part 2 of a PR trilogy which removes graphql dependency from the frontend

In this PR we define the required interface that the frontend needs to call then we use the generated axum router to merge it with the router in the /user path of the authentication service.

Please review commit by commit

Screenshots, GIFs, and Videos

@seanaye seanaye requested a review from a team as a code owner December 3, 2025 21:30
@seanaye seanaye changed the base branch from main to seanaye/add/axum-rpc-macro December 3, 2025 21:34
@seanaye seanaye force-pushed the seanaye/add/legacy-rpc-interface branch from 78f12ee to bb12f54 Compare December 3, 2025 21:36
@seanaye seanaye force-pushed the seanaye/add/axum-rpc-macro branch from a312fae to 25387b6 Compare December 3, 2025 21:36
@seanaye seanaye force-pushed the seanaye/add/legacy-rpc-interface branch from bb12f54 to 602e208 Compare December 3, 2025 21:41
@seanaye seanaye force-pushed the seanaye/add/axum-rpc-macro branch from 25387b6 to e7408ae Compare December 3, 2025 21:41
@seanaye seanaye force-pushed the seanaye/add/legacy-rpc-interface branch from 602e208 to 1ef9e07 Compare December 4, 2025 15:00
@seanaye seanaye force-pushed the seanaye/add/axum-rpc-macro branch from e7408ae to 1da870a Compare December 4, 2025 15:00
Base automatically changed from seanaye/add/axum-rpc-macro to main December 4, 2025 19:31
@seanaye seanaye force-pushed the seanaye/add/legacy-rpc-interface branch 3 times, most recently from 9574e31 to 1f9eed0 Compare December 4, 2025 19:35
@seanaye seanaye force-pushed the seanaye/add/legacy-rpc-interface branch 2 times, most recently from 12aeab4 to b74c504 Compare December 4, 2025 20:03
@seanaye seanaye force-pushed the seanaye/add/legacy-rpc-interface branch from b74c504 to eca3364 Compare December 4, 2025 20:55
@seanaye seanaye merged commit b30d999 into main Dec 5, 2025
36 checks passed
@seanaye seanaye deleted the seanaye/add/legacy-rpc-interface branch December 5, 2025 14:55
nickisnoble pushed a commit that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants