Skip to content

Replace deprecated code-placeholders shortcode with placeholders attribute#6869

Merged
jstirnaman merged 2 commits intoritwika/query_logfrom
copilot/sub-pr-6866
Feb 25, 2026
Merged

Replace deprecated code-placeholders shortcode with placeholders attribute#6869
jstirnaman merged 2 commits intoritwika/query_logfrom
copilot/sub-pr-6866

Conversation

Copy link
Contributor

Copilot AI commented Feb 25, 2026

Addresses review feedback to modernize placeholder syntax in query log documentation.

Changes

  • Removed deprecated {{% code-placeholders "DATABASE_TOKEN" %}} wrapper shortcode
  • Added { placeholders="DATABASE_TOKEN" } attribute directly to code fences

Before

{{% code-placeholders "DATABASE_TOKEN" %}}

```sh
influxctl query \
  --token DATABASE_TOKEN \
  --database _internal
```

{{% /code-placeholders %}}

After

```sh { placeholders="DATABASE_TOKEN" }
influxctl query \
  --token DATABASE_TOKEN \
  --database _internal
```

The modern attribute-based approach is more concise and aligns with current documentation standards per DOCS-SHORTCODES.md.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…ibute

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP Address feedback on query log documentation Replace deprecated code-placeholders shortcode with placeholders attribute Feb 25, 2026
Copilot AI requested a review from jstirnaman February 25, 2026 14:07
@jstirnaman jstirnaman marked this pull request as ready for review February 25, 2026 14:08
@jstirnaman jstirnaman requested a review from a team as a code owner February 25, 2026 14:08
@jstirnaman jstirnaman requested review from sanderson and removed request for a team February 25, 2026 14:08
@jstirnaman jstirnaman merged commit 8e32328 into ritwika/query_log Feb 25, 2026
1 check was pending
jstirnaman added a commit that referenced this pull request Feb 25, 2026
* Add documentation for viewing the query log in the Admin UI and using influxctl

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Replace deprecated code-placeholders shortcode with placeholders attribute (#6869)

* Initial plan

* Replace deprecated code-placeholders shortcode with placeholders attribute

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

* Fix support.influxdata.com link pattern in query-log.md (#6867)

* Initial plan

* Fix support link pattern and add enforcement check

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

* Remove validation changes, keep content fix only

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

Co-authored-by: Jason Stirnaman <jstirnaman@influxdata.com>

* Apply suggestions from code review

Co-authored-by: Jason Stirnaman <jstirnaman@influxdata.com>

* Clarify note on query log enabling and its impact on cluster load per @jeffreyssmith2nd 's suggestion

* Add schema table with explicit data types for query_log fields (#6868)

* Initial plan

* Add comprehensive schema table for query_log fields with data types and descriptions

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

* Update content/influxdb3/cloud-dedicated/query-data/troubleshoot-and-optimize/query-log.md

---------

Co-authored-by: Jason Stirnaman <jstirnaman@influxdata.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jstirnaman <212227+jstirnaman@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.

2 participants