Skip to content

Dev 7574 generate graphql client#58

Merged
charlta merged 32 commits intomainfrom
DEV-7574-Generate-Graphql-boilerplate
Apr 1, 2026
Merged

Dev 7574 generate graphql client#58
charlta merged 32 commits intomainfrom
DEV-7574-Generate-Graphql-boilerplate

Conversation

@chestm007
Copy link
Copy Markdown
Contributor

Description

GraphQL queries are now properly typed objects, and all parameters to the query are also that.

EasClient:
major rewrite...
updated to operate in async mode or not via init arg.
accepts kwargs only in init
session and json_serializer have been nuked. session is not needed (AFAICT), can bring back json_serializer if we really want too.

Gql objects:
work_package.py and the likes have been nuked.
Every object is generated via Introspection of an EAS API.

Associated tasks

None

Test Steps

All the things, probs connect to staging and run them all. SHOULD just work ™️

Checklist

If any of these are not applicable, strikethrough the line ~like this~. Do not delete it!. Let the reviewer decide if you should have done it.

Code

  • I have performed a self review of my own code (including checking issues raised when creating the PR).
  • I have added/updated unit tests for these changes, and if not I have explained why they are not necessary.
  • I have commented my code in any hard-to-understand or hacky areas.
  • I have handled all new warnings generated by the compiler or IDE.
  • I have rebased onto the target branch (usually main).

Security

When developing applications, use following guidelines for information security considerations:

  • Access to applications should be protected with security keys/tokens or usernames and passwords;
  • All sessions are encrypted if possible;
  • All application input is sanitised before being acted on (ie SQL statements, etc);
  • Log messages, and especially client-facing ones, must be handled securely and must not leak credentials information (internal URLs, passwords, tokens).
  • I have considered if this change impacts information security and made sure those impacts are handled.

Documentation

  • I have updated the changelog.
  • I have updated any documentation required for these changes.

Breaking Changes

  • I have considered if this is a breaking change and will communicate it with other team members by posting it on the Slack breaking-changes channel.

Please leave a summary of the breaking changes here and then post it on the Slack breaking-changes channel to notify the team about it.

Tests are still a bit poo though.

Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
…nerated client into zepben.eas

Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
@ryanjcoleman
Copy link
Copy Markdown
Member

Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
…tuff too

Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
@chestm007 chestm007 requested a review from charlta March 26, 2026 07:34
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
charlta added 3 commits March 30, 2026 15:05
Signed-off-by: Anthony Charlton <anthony.charlton@zepben.com>
Signed-off-by: Anthony Charlton <anthony.charlton@zepben.com>
Signed-off-by: Anthony Charlton <anthony.charlton@zepben.com>
charlta
charlta previously approved these changes Mar 30, 2026
charlta added 5 commits March 30, 2026 15:58
Signed-off-by: Anthony Charlton <anthony.charlton@zepben.com>
Signed-off-by: Anthony Charlton <anthony.charlton@zepben.com>
Signed-off-by: Anthony Charlton <anthony.charlton@zepben.com>
Signed-off-by: Anthony Charlton <anthony.charlton@zepben.com>
Signed-off-by: Anthony Charlton <anthony.charlton@zepben.com>
@charlta charlta self-requested a review March 30, 2026 06:47
@charlta charlta marked this pull request as draft March 30, 2026 06:47
@charlta
Copy link
Copy Markdown
Member

charlta commented Mar 30, 2026

Converted to draft for typing rework

Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
…ode gen lib

Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
Signed-off-by: Max Chesterfield <max.chesterfield@zepben.com>
@charlta charlta marked this pull request as ready for review April 1, 2026 13:56
@charlta charlta merged commit af13a51 into main Apr 1, 2026
4 of 5 checks passed
@charlta charlta deleted the DEV-7574-Generate-Graphql-boilerplate branch April 1, 2026 13:56
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.

3 participants