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

Generate a better openapi client that allows us to send zero values, … #1958

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

chirino
Copy link
Member

@chirino chirino commented Mar 6, 2024

…instead of always omitting them.

  • consolidate client bits in the client package
  • use the new /api/events endpoint
  • make device bearer tokens work even if the device is connected with oauth tokens.

…instead of always omitting them.

* consolidate client bits in the client package
* use the new /api/events endpoint
* make device bearer tokens work even if the device is connected with oauth tokens.

Signed-off-by: Hiram Chirino <hiram@hiramchirino.com>
Copy link

netlify bot commented Mar 6, 2024

Deploy Preview for nexodus-docs canceled.

Name Link
🔨 Latest commit fb3bfac
🔍 Latest deploy log https://app.netlify.com/sites/nexodus-docs/deploys/65e8a509c15fbc0008f483d7

@mergify mergify bot added missing-docs-label missing has-docs or no-docs-needed missing-design-label missing has-design or no-design-needed label missing-tests-label This PR is missing a tests label, has-tests or no-tests-required labels Mar 6, 2024
@chirino chirino added no-design-required This PR does not require a design document no-docs-required This PR does not require user facing docs. has-tests PR includes test coverage and removed missing-docs-label missing has-docs or no-docs-needed missing-design-label missing has-design or no-design-needed label missing-tests-label This PR is missing a tests label, has-tests or no-tests-required labels Mar 6, 2024
@mergify mergify bot merged commit 5f6a8ba into nexodus-io:main Mar 6, 2024
29 checks passed
github-actions bot pushed a commit that referenced this pull request Mar 6, 2024
Merge pull request #1958 from chirino/better-openapi-client

Generate a better openapi client that allows us to send zero values, …
@chirino chirino deleted the better-openapi-client branch July 23, 2024 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has-tests PR includes test coverage no-design-required This PR does not require a design document no-docs-required This PR does not require user facing docs.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants