Skip to content

Conversation

mortik
Copy link
Member

@mortik mortik commented Jan 25, 2023

Shop Data gets loaded via GraphQL and mapped againts current models in the DB. Result is filtered for Paint Packs and not resolved models.

end

defp graphql_endpoint do
"https://robertsspaceindustries.com/graphql"
Copy link
Member

Choose a reason for hiding this comment

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

This could be a config option. either directly the full URI, or host as one option (probably global) and path as second option

[
{:ex_fleet_yards, in_umbrella: true}
{:ex_fleet_yards, in_umbrella: true},
{:neuron, "~> 5.0.0"}
Copy link
Member

Choose a reason for hiding this comment

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

needs an update to the nix/mix.nix file to not break nix builds.

Copy link
Member Author

Choose a reason for hiding this comment

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

is there a task to regenerate the mix.nix?

Copy link
Member

Choose a reason for hiding this comment

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

there is a task, but currently seems broken, mix2nix > nix/mix.nix is how I generated it (having mix2nix installed via the devenv)

end

defp load_models(name, mapping) do
from(m in Game.Model,
Copy link
Member

Choose a reason for hiding this comment

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

Could be moved to ExFleetYards.Repo.Game.Model as function there, but also fine here.

@mortik mortik force-pushed the feat/rsi-paints-import branch from 002dd28 to fb123ae Compare January 25, 2023 23:21
Shop Data gets loaded via GraphQL and mapped againts current models in the DB.
Result is filtered for Paint Packs and not resolved models.
@mortik mortik force-pushed the feat/rsi-paints-import branch from fb123ae to 3e7e0a4 Compare February 5, 2023 22:53
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