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

Update functions tables docs #17071

Merged
merged 2 commits into from Mar 11, 2024
Merged

Update functions tables docs #17071

merged 2 commits into from Mar 11, 2024

Conversation

car12o
Copy link
Contributor

@car12o car12o commented Feb 27, 2024

Summary

Update functions tables docs

@Ancairon
Copy link
Member

Ancairon commented Mar 5, 2024

@car12o this file is not on Learn currently, I get this is a docs update and I'm reviewer, do you want to add it in Learn?

@car12o
Copy link
Contributor Author

car12o commented Mar 5, 2024

@car12o this file is not on Learn currently, I get this is a docs update and I'm reviewer, do you want to add it in Learn?

I don't think we should add this to learn for now.
Functionality is not yet available for users.

Copy link
Member

@Ancairon Ancairon left a comment

Choose a reason for hiding this comment

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

just a small tweak because yellow is not visible in light-mode

Comment on lines 129 to 131
### ${\color{yellow}IMPORTANT}$

On Data values, `timestamp` column value must be in unix micro.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
### ${\color{yellow}IMPORTANT}$
On Data values, `timestamp` column value must be in unix micro.
> **Important Note**
>
> On Data values, `timestamp` column value must be in unix micro.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed, although I think it will remove the emphasis it should have

Copy link
Member

Choose a reason for hiding this comment

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

If you want to have it as it was go for it, if this is not in Learn I won't gatekeep it, it's just that yellow was not looking good, and the preview style was odd to me

@car12o car12o merged commit e1411d2 into master Mar 11, 2024
138 of 139 checks passed
@car12o car12o deleted the docs/update-functions-tables branch March 11, 2024 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/collectors Everything related to data collection area/docs collectors/plugins.d
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants