Skip to content

Conversation

@syvb
Copy link
Member

@syvb syvb commented Jan 31, 2023

Description

For the state/timeline aggregates, the name of the argument for the aggregate is written as tws instead of agg in a few parts of the documentation. tws stands for Time-Weighted Summary, which is not applicable here. I think I copied from the time-weighted average documentation and forgot to change the parameter name.

Writing help

For information about style and word usage, see the style guide

Review checklists

Reviewers: use this section to ensure you have checked everything before approving this PR:

Subject matter expert (SME) review checklist

  • Is the content technically accurate?
  • Is the content complete?
  • Is the content presented in a logical order?
  • Does the content use appropriate names for features and products?
  • Does the content provide relevant links to further information?

Documentation team review checklist

  • Is the content free from typos?
  • Does the content use plain English?
  • Does the content contain clear sections for concepts, tasks, and references?
  • Have any images been uploaded to the correct location, and are resolvable?
  • If the page index was updated, are redirects required
    and have they been implemented?
  • Have you checked the built version of this content?

@github-actions
Copy link

Allow 10 minutes from last push for the staging site to build. If the link doesn't work, try using incognito mode instead. For internal reviewers, check web-documentation repo actions for staging build status. Link to build for this PR: http://docs-dev.timescale.com/docs-sv-tws-name-fix

@syvb syvb merged commit e911ceb into latest Feb 1, 2023
@syvb syvb deleted the sv/tws-name-fix branch February 1, 2023 19:43
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