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

Issue/59 #61

Merged
merged 19 commits into from
Jun 19, 2024
Merged

Issue/59 #61

merged 19 commits into from
Jun 19, 2024

Conversation

thibault-cne
Copy link
Member

@thibault-cne thibault-cne commented Jun 19, 2024 •
edited
Loading

Description

Changed the API style from RESTfull to GraphQL. All tests have been ported, rate limiting middleware and caching middlewares have been deleted to prepare for the microservices architecture.

  • ported every endpoint to a GraphQL graph
  • ported every tests to work with graphql
  • removed rate limiting and caching middlewares
  • added graphql configuration to the config file
  • renamed necessary files and struct to meet the new project name

Related Issue

Issue #59

Motivation and Context

How Has This Been Tested?

Checklist

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes/features)
  • Docs have been added / updated (for bug fixes / features)

thibault-cne added 19 commits May 23, 2024 10:54
@thibault-cne
feat: started porting the API to graphQL
d83b35b
@thibault-cne
feat: ported the following to graphQL -> Circuits, Races, Constructor…
603918f 
…Standings, Constructors
@thibault-cne
feat: addded the graphQL support for driver standings
a6bc00a
@thibault-cne
feat: addded the graphQL support for drivers
05d3521
@thibault-cne
feat: addded the graphQL support for laps
85c125b
@thibault-cne
feat: addded the graphQL support for pit stops
5c7ce7a
@thibault-cne
feat: addded the graphQL support for seasons
4c8cf51
@thibault-cne
feat: addded the graphQL support for status
e9bfec5
@thibault-cne
feat: ported all tests to graphQL
130c215
@thibault-cne
feat: edited all files with the new repo name
7888d0b
@thibault-cne
feat: edited readme
7fd6474
@thibault-cne
feat: edited readme
9e2c9f9
@thibault-cne
fix: fixed circleci config
554a137
@thibault-cne
feat: edited readme
d1dae5a
@thibault-cne
feat: changed the name of the main struct
8f8c16b
@thibault-cne
feat: removed all unsage unwrap and fixed pagination system
aeb9d44
@thibault-cne
fix: fixed tests
d355526
@thibault-cne
fix: fixed tests
b00f278
@thibault-cne
feat: removed rate limiter and caching middlewares and adding graphiq…
c6327e9 
…l endpoint in the config
@thibault-cne thibault-cne added enhancement Modify an existing feature feature Need a new feature labels Jun 19, 2024
@thibault-cne thibault-cne added this to the Version v0.2.0 milestone Jun 19, 2024
@thibault-cne thibault-cne self-assigned this Jun 19, 2024
@thibault-cne thibault-cne linked an issue Jun 19, 2024 that may be closed by this pull request
Take a look into graphQL #59
Closed
1 task
@thibault-cne thibault-cne merged commit 780ff25 into master Jun 19, 2024
4 checks passed
@thibault-cne thibault-cne deleted the issue/59 branch June 19, 2024 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@thibault-cne thibault-cne

Labels
enhancement Modify an existing feature feature Need a new feature
Projects
None yet
Milestone
Version v0.2.0
Development

Successfully merging this pull request may close these issues.

Take a look into graphQL
1 participant
@thibault-cne