Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure codegen script accommodates all subgraph types (eg derived fields) #7

Open
2 tasks
SvenMuller95 opened this issue Aug 19, 2022 · 3 comments
Open
2 tasks
Assignees

Comments

@SvenMuller95
Copy link
Collaborator

SvenMuller95 commented Aug 19, 2022

  • research subgraph types (currently 6 types)
  • ensure tool is accommodating for all
@MJYoung114
Copy link
Contributor

need to handle derivedFrom types -> suggestion is to ignore these in the generated code for an entity, The field will be set in the derived from entity https://thegraph.com/docs/en/developing/creating-a-subgraph/#reverse-lookups

@MJYoung114
Copy link
Contributor

  • Arrays of entities
  • arrays of types
  • bigDecimal

@JasoonS JasoonS closed this as completed Dec 14, 2022
@JasoonS JasoonS reopened this Dec 26, 2022
@JasoonS JasoonS changed the title Ensure codegen script accommodates all subgraph types Ensure codegen script accommodates all subgraph types (eg derived fields) Dec 26, 2022
@JasoonS
Copy link
Member

JasoonS commented Dec 26, 2022

I think in the process of doing this task more type documentation and structuring can be done in the code. Currently almost everything is polymorphic. It might be good to use less polymorphic structures in the code.

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

No branches or pull requests

3 participants