kellpossible / kellpossible/coster

Switch to updated graphql web client when it's ready/merged

Open
#22 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
13
Forks
0
PR merge metrics
No merged PRs in 30d

Description

According to the discussion in https://github.com/graphql-rust/graphql-client/issues/331 there should hopefully be a new graphql web client implementation arriving in pull request https://github.com/graphql-rust/graphql-client/pull/327 which relies on reqwest. When the PR is merged it would be good to remove the custom port of this module in gui::graphql::client.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Check the upstream discussion in graphql-client issue 331 and pull request 327 to confirm the reqwest-based web client is available. Then inspect the custom module at gui::graphql::client; done means removing that port and switching to the updated client.

Written by the indexing model from the issue text.

Assessment

Tech stack
graphql, rust
Domain
api
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.