Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

Merge the mas_graphql crate into the mas_handlers crate #2783

Merged
merged 5 commits into from
May 17, 2024

Conversation

reivilibre
Copy link
Contributor

This one is to be squash merged, it's only presented as separate non-atomic commits for easier review
(I personally like to have the rename-only operation, as git recognises it, as one commit)

Copy link

cloudflare-workers-and-pages bot commented May 17, 2024

Deploying matrix-authentication-service-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: de590a3
Status: ✅  Deploy successful!
Preview URL: https://00ca41e0.matrix-authentication-service-docs.pages.dev
Branch Preview URL: https://rei-graphql-into-handlers.matrix-authentication-service-docs.pages.dev

View logs

@reivilibre reivilibre force-pushed the rei/graphql_into_handlers branch from d6ebae5 to a1d6be0 Compare May 17, 2024 10:53
@reivilibre reivilibre force-pushed the rei/graphql_into_handlers branch from a1d6be0 to 9d5fcf3 Compare May 17, 2024 11:50
@reivilibre reivilibre force-pushed the rei/graphql_into_handlers branch from 9d5fcf3 to 60ed1a1 Compare May 17, 2024 12:14
@reivilibre reivilibre marked this pull request as ready for review May 17, 2024 12:19
@reivilibre reivilibre requested a review from sandhose May 17, 2024 12:19
Copy link
Member

@sandhose sandhose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks sane overall, thanks!

crates/handlers/src/graphql/mod.rs Outdated Show resolved Hide resolved
crates/handlers/src/graphql/mod.rs Outdated Show resolved Hide resolved
crates/handlers/src/graphql/mod.rs Show resolved Hide resolved
crates/handlers/src/graphql/model/oauth.rs Show resolved Hide resolved
crates/handlers/src/graphql/mod.rs Outdated Show resolved Hide resolved
@reivilibre reivilibre requested a review from sandhose May 17, 2024 15:41
@reivilibre reivilibre merged commit 206d45b into main May 17, 2024
16 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants