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

refactor(influxdb): Refactor InfluxDBv1+v2 functions #30

Merged
merged 1 commit into from
Jun 8, 2023

Conversation

Slider0007
Copy link
Owner

@Slider0007 Slider0007 commented Jun 8, 2023

  • Add some error handling + more verbose logs
  • Rename existing debug logs
  • Rerrange some functions to align v1 and v2 functions
  • Check against parameter value undefined -> avoid exception (e.g. if URI is set to undefined)
  • Return false on init if any parameter is not valid (v1: URI, Database | v2: URI, Database, Org, Token)
    • empty or equal to undefined

BEGIN_COMMIT_OVERRIDE
refactor(influxdb): Refactor InfluxDBv1+v2 functions
END_COMMIT_OVERRIDE

@Slider0007 Slider0007 merged commit 4f59958 into develop Jun 8, 2023
@Slider0007 Slider0007 deleted the refactor-influxdb branch June 9, 2023 17:29
@Slider0007 Slider0007 changed the title Refactor InfluxDBv1+v2 functions refactor(influxdb): Refactor InfluxDBv1+v2 functions Sep 7, 2023
Slider0007 pushed a commit that referenced this pull request Oct 17, 2024
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.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.

1 participant