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.

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

chore: Update swagger #84

Merged

Conversation

antoinegelloz
Copy link
Contributor

@antoinegelloz antoinegelloz commented Jan 9, 2023

  • fixed swagger mistakes
  • changed swagger to have cleaner generated structs, better naming, more consistent with other services
  • removed client generation
  • make sure pre-commit has run

@codecov
Copy link

codecov bot commented Jan 9, 2023

Codecov Report

Merging #84 (d66f0c2) into feat/switch-to-cursor-pagination (f18b489) will decrease coverage by 0.10%.
The diff coverage is 0.00%.

@@                         Coverage Diff                          @@
##           feat/switch-to-cursor-pagination      #84      +/-   ##
====================================================================
- Coverage                             18.50%   18.39%   -0.11%     
====================================================================
  Files                                   103      103              
  Lines                                  4674     4674              
====================================================================
- Hits                                    865      860       -5     
- Misses                                 3710     3714       +4     
- Partials                                 99      100       +1     
Impacted Files Coverage Δ
internal/app/api/payments.go 0.00% <ø> (ø)
internal/app/migrations/003_tasks.go 5.00% <0.00%> (ø)
internal/app/migrations/005_payments.go 2.19% <0.00%> (ø)
main.go 0.00% <ø> (ø)
...rnal/app/connectors/dummypay/task_generate_file.go 67.28% <0.00%> (-4.68%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@antoinegelloz antoinegelloz merged commit c581866 into feat/switch-to-cursor-pagination Jan 9, 2023
@antoinegelloz antoinegelloz deleted the chore/update-swagger branch January 9, 2023 09:18
darkmatterpool added a commit that referenced this pull request Jan 11, 2023
* feat: cursor-based pagination for payments and tasks

Signed-off-by: Lawrence Zawila <113581282+darkmatterpool@users.noreply.github.com>

* fix: swagger definitions & accounts cursor support

Signed-off-by: Lawrence Zawila <113581282+darkmatterpool@users.noreply.github.com>

* fix: swagger cursor description

Signed-off-by: Lawrence Zawila <113581282+darkmatterpool@users.noreply.github.com>

* chore: Update swagger (#84)

* feat: added pagination tests

Signed-off-by: Lawrence Zawila <113581282+darkmatterpool@users.noreply.github.com>

* fix: test cases

Signed-off-by: Lawrence Zawila <113581282+darkmatterpool@users.noreply.github.com>

Signed-off-by: Lawrence Zawila <113581282+darkmatterpool@users.noreply.github.com>
Co-authored-by: Antoine Gelloz <antoine.gelloz@me.com>
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