search

beethovenxfi / beethovenx-backend

MIT License
13 stars 15 forks source link

only have one set of resolvers and schema, keep folders for dev #538

Closed franzns closed 11 months ago

franzns commented 11 months ago

addresses #523

gmbronco commented 11 months ago

btw. would it make sense to merge loadRestRoutesBeethoven + loadRestRoutesBalancer as well?

franzns commented 11 months ago

btw. would it make sense to merge loadRestRoutesBeethoven + loadRestRoutesBalancer as well?

I thought about it too, but it's not as important. I wanted to merge the schema because you could run into issues where a certain schema is deployed but not available. Can't happen with the restRoutes as they dont use codegen

That being said, might nevertheless make sense. I'm going to add it.