Skip to content
This repository has been archived by the owner on Aug 27, 2023. It is now read-only.

Chain resolvers to learn query #14

Merged
merged 4 commits into from
Mar 14, 2023
Merged

Chain resolvers to learn query #14

merged 4 commits into from
Mar 14, 2023

Conversation

Camilo-Camargo
Copy link
Member

@Camilo-Camargo Camilo-Camargo commented Mar 5, 2023

Added chaining resolvers from learners to learning files, improving the structure and organization of our codebase. However, during testing, we discovered a problem with the same field resolvers for events and activities. Specifically, both contain learning files as a field, which is causing issues with our query. We will need to investigate this further and determine the best approach to resolve this conflict.

Edition from Octo GitHub nvim client.

The TypeGrahQL community problem MichalLytek/type-graphql#193.

Example

query learner {
  learner {
    email
    events {
      id
      title
      endDate
      activity {
        id
        endTime
        learningFile {
          id
          format
        }
      }
    }
  }
}

@Camilo-Camargo Camilo-Camargo requested a review from a user March 5, 2023 17:57
@Camilo-Camargo Camilo-Camargo enabled auto-merge March 5, 2023 17:58
@Camilo-Camargo Camilo-Camargo disabled auto-merge March 5, 2023 18:02
@Camilo-Camargo Camilo-Camargo enabled auto-merge March 7, 2023 02:18
@Camilo-Camargo Camilo-Camargo merged commit 2330032 into main Mar 14, 2023
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.

1 participant