Skip to content

use graphql files for documents#289

Merged
kdembler merged 2 commits intoJoystream:personalisationfrom
kdembler:graphql-cleanup
Feb 1, 2021
Merged

use graphql files for documents#289
kdembler merged 2 commits intoJoystream:personalisationfrom
kdembler:graphql-cleanup

Conversation

@kdembler
Copy link
Copy Markdown
Contributor

@kdembler kdembler commented Feb 1, 2021

No description provided.

Copy link
Copy Markdown
Contributor

@drillprop drillprop left a comment

Choose a reason for hiding this comment

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

I'm not sure about these single-line files inside the queries folder. Besides that, looks good to me

}
}
`
export * from './__generated__/channels.generated'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we really need these files? Can we just create one index.ts file?

@kdembler kdembler merged commit c945786 into Joystream:personalisation Feb 1, 2021
kdembler added a commit that referenced this pull request Mar 2, 2021
* use graphql files for documents

* export from single file
kdembler added a commit that referenced this pull request Mar 2, 2021
* use graphql files for documents

* export from single file
@kdembler kdembler deleted the graphql-cleanup branch September 29, 2022 11:05
This was referenced Oct 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants