legumeinfo / graphql-server

A GraphQL server that supports the Legume Information System and related biodata websites
Apache License 2.0
2 stars 3 forks source link

5.1.0.4 model changes #112

Open alancleary opened 8 months ago

alancleary commented 8 months ago

@sammyjava has a 5.1.0.4-model-changes branch that adds support for Intermine version 5.1.0.4, which is required to integrate the trait association search web component into the LIS site, among other things. I'm opening this issue to serve as a reference for things related to this work that are located in different repos. Edit: Sam has left the project so this issue is being repurposed to keep track of the work that needs to be done to finish implementing support for InterMine version 5.1.0.4.

sammyjava commented 7 months ago

I've done some housekeeping/streamlining of the 5.1.0.4 model. It's not that much different, but note to self to double-check all the types and resolvers and queries against 5.1.0.4.

sammyjava commented 6 months ago

I've renamed the interfaces to FooInterface with a resolvable type Foo that implements it. I've added a boatload of types that hadn't been added yet.

Generally, if a field like locatedFeatures originates in a class like BioEntity (it does), then the resolver will treat the object as a BioEntity for resolution. This helps simplify some things like resolving locatedFeatures, which was missing before.

sammyjava commented 5 months ago

I've done the final updates to the types in the 5.1.0.4-model-changes branch. However, work needs to be done with resolvers to handle a few late things, if one wants to get 5.1.0.4 going with the old code base in this repo, rather than rolling out the big has-stuff-interfaces branch, which has all of the below done plus a helluva lot more. I may be able to get that work done next Thursday so 5.1.0.4 is tested and ready to go.