Skip to content

Conversation

sanderson
Copy link
Collaborator

  • Adds a new EU wind sample data set
  • Adds new sample-data-dist.md for Cloud Dedicated and Clustered sample data pages
  • Corrects v3 write API instructions for writing sample data to for Core and Enterprise
  • Adds influxctl write instructions for writing sample data to Cloud Dedicated and Clustered
  • Fixes a JS bug that caused the URL selector button to be appended to a code block twice
  • Fixed JS linting issues

  • Rebased/mergeable

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a new European Union wind sample dataset to the InfluxDB v3 documentation and improves the codebase structure. The changes include updating v3 write API examples to use proper HTTP methods and headers, consolidating documentation using shared content files, and fixing JavaScript issues with URL selector buttons.

Key Changes

  • Adds EU wind sample dataset with hierarchical geographic tags (country, county, city)
  • Updates API examples with proper HTTP methods and standardized headers across all endpoints
  • Refactors shared content to reduce duplication between Cloud Dedicated and Clustered documentation

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
content/shared/influxdb3-sample-data/sample-data.md Adds EU wind dataset and standardizes v3 API examples with proper HTTP methods
content/shared/influxdb3-sample-data/sample-data-dist.md New shared content file for Cloud Dedicated/Clustered with influxctl CLI examples
content/influxdb3/clustered/reference/sample-data.md Converts to use shared content source
content/influxdb3/cloud-dedicated/reference/sample-data.md Converts to use shared content source
content/influxdb3/cloud-serverless/reference/sample-data.md Adds EU wind dataset section
assets/styles/layouts/article/_buttons.scss Adjusts URL selector button positioning
assets/js/influxdb-url.js Fixes URL selector duplication bug and improves code formatting

Copy link
Contributor

@jstirnaman jstirnaman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Just some comment cleanup and addition (stuff I forget and then have to lookup).


// Retrieve the previously selected URLs from the from the urls local storage object.
// This is used to update URLs whenever you switch between browser tabs.
// Retrieve the previously selected URLs from the from the urls local storage
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// Retrieve the previously selected URLs from the from the urls local storage
// Retrieve the previously selected URLs from the urls local storage

//////////////////////////////////////////////////////////////////////////////

// Validate custom URLs
function validateUrl(url) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
function validateUrl(url) {
/** Match 3 possible types of hosts:
* Named host = (unreserved | pct-encoded | sub-delims)+
* IPv6 host = \[([a-f0-9:.]+)\]
* IPvFuture host = \[v[a-f0-9][a-z0-9\-._~%!$&'()*+,;=:]+\]
* Port = [0-9]+
*/

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.

2 participants