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.

Sign up for GitHub

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

Jump to bottom

Remove mirror usage. Refactor to be code generation friendly. #7

Merged
merged 16 commits into from
Aug 25, 2017
Merged

Remove mirror usage. Refactor to be code generation friendly. #7

merged 16 commits into from
Aug 25, 2017

Conversation

hourliert
Copy link
Member

@hourliert hourliert commented Aug 23, 2017 •
edited
Loading

Fixes #3 #5 .

  • remove mirror usage
  • create a custom DSL that is code generation friendly
  • lighten the client code (250 LOC)
  • handle API GQL errors

TODO:

  • remove mirror usage
  • create a custom DSL that is code generation friendly
  • handle errors
  • review public / private members
  • document every public members
  • update the roadmap
  • explain how the code generator should work

@hourliert
refactor(*): change the whole behavior
ff1fb4f 
* remove mirror usage
* create custom DSL code generation friendly
* lighten the client code (250 LOC)

TODO:
* review public / private member
* document everything
* update the roadmap
* explain how the code generator should work
@hourliert hourliert added the WIP label Aug 23, 2017
@hourliert hourliert self-assigned this Aug 23, 2017
This was referenced Aug 23, 2017
Closed
Closed
hourliert added 15 commits August 23, 2017 13:59
@hourliert
chore(pubspec): remove uuid dep
d2f2300
@hourliert
refactor(encoder): clean up code
d7edc29
@hourliert
feat(*): add error handling
d72c88c 
TODO: handle GQL error from API
@hourliert
feat(client): handle gql error returned by API
4014305 
clean up error handling
@hourliert
refactor(graphql_definitions): create GQLField and GQLOperation inter…
f284095 
…faces
@hourliert
refactor(graphql_definitions): create Fields and Fragments mixins
a9e93a7 
Reduce queries definitions size.
@hourliert
refactor(query_examples): re order defintions
ff79c68
@hourliert
refactor(graphql): rename graphql_defintions into graphql
2ac0999
@hourliert
refactor(alias): make aliasId private
476c69c
@hourliert
chore(*): enforce linter
baa02dc
@hourliert
chore(*): enforce linter rules bis
bb830c8
@hourliert
doc(*): documente every public member
2019d53
@hourliert
doc(*): update README ROADMAP and CHANGELOG
fee547f
@hourliert
refactor(graphql_client): split library in 2: client and dsl
74dfac8
@hourliert
doc(code_gen): create a document that details the code generator
6715bcb
@hourliert hourliert force-pushed the refactor/remove_mirrors branch from 2943403 to 6715bcb Compare August 25, 2017 13:02
@hourliert hourliert merged commit f73280c into master Aug 25, 2017
@hourliert hourliert deleted the refactor/remove_mirrors branch August 25, 2017 13:05
@hourliert hourliert mentioned this pull request Aug 27, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@hourliert hourliert

Labels
WIP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hourliert