Conversation
- Introduced the AuthenticatedUser type in both API schemas, enhancing user data representation. - Implemented a resolver for AuthenticatedUser to handle user retrieval and fallback for missing first names. - Updated user object exports to include AuthenticatedUser for better federation support.
- Added the AuthenticatedUser type to the API schemas, including fields for user details. - Extended the AuthenticatedUser type to support federation with external services. - Updated generated TypeScript definitions to reflect the new structure of AuthenticatedUser. - Modified user-related GraphQL operations to incorporate the new AuthenticatedUser type.
- Updated the AuthenticatedUser type in the API schemas to include federation capabilities with the API_JOURNEYS_MODERN graph. - Introduced a new external reference for AuthenticatedUser in the user schema to facilitate integration. - Modified TypeScript exports to include the new AuthenticatedUser reference for improved type handling.
- Deleted the AuthenticatedUser type definition from the GraphQL schema to streamline user data representation. - This change simplifies the API by removing unnecessary complexity related to user authentication details.
- Added languageUserRoles field to the AuthenticatedUser type in both schema.graphql and schema.graphql for API_LANGUAGES. - Implemented resolver for languageUserRoles in the user schema to fetch user-specific language roles from the database. - This update improves the user model by integrating language role management into the authentication process.
WalkthroughAdds AuthenticatedUser as a federated entity across multiple subgraphs (API_JOURNEYS, API_JOURNEYS_MODERN, API_LANGUAGES, API_USERS), introduces LanguageRole enum in API_LANGUAGES, and wires externalRefs/implementations and _Entity updates so the gateway exposes the extended AuthenticatedUser type. Changes
Sequence Diagram(s)mermaid Client->>Gateway: Query AuthenticatedUser (fields including languageUserRoles) Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
View your CI Pipeline Execution ↗ for commit f679297
☁️ Nx Cloud last updated this comment at |
|
View your CI Pipeline Execution ↗ for commit f679297
☁️ Nx Cloud last updated this comment at |
|
View your CI Pipeline Execution ↗ for commit f679297 ☁️ Nx Cloud last updated this comment at |
|
The latest updates on your projects.
|
|
The latest updates on your projects.
|
|
The latest updates on your projects.
|
|
The latest updates on your projects.
|
|
The latest updates on your projects.
|
|
The latest updates on your projects.
|
|
The latest updates on your projects.
|
Summary by CodeRabbit
Refactor
New Features