-
Notifications
You must be signed in to change notification settings - Fork 10.2k
Fix flowtrackd naming in schema map #22978
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
Fix flowtrackd naming in schema map #22978
Conversation
|
This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 To request another review, post a new comment with "/windsurf-review".
| In this example, we are going to use the GraphQL Analytics API to query for Firewall Events over a specified time period. | ||
|
|
||
| The following API call will request Firewall Events over a one hour period, and output the requested fields. Be sure to replace `<CLOUDFLARE_ZONE_TAG>`, `<EMAIL>`, and `<API_TOKEN>`[^1] with your zone tag and API credentials, and adjust the `datetime_geg` and `datetime_leq` values to your liking. | ||
| The following API call will request Firewall Events over a one hour period, and output the requested fields. Be sure to replace `<Cloudflare_ZONE_TAG>`, `<EMAIL>`, and `<API_TOKEN>`[^1] with your zone tag and API credentials, and adjust the `datetime_geq` and `datetime_leq` values to your liking. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a typo in the parameter name in the text. It says 'datetime_geg' in the original text which was corrected to 'datetime_geq' in this PR, matching the actual parameter used in the code example.
| <th>flowtrackedNetworkAnalytics-AdaptiveGroups</th> | ||
| <th>magicFirewallNetworkAnalytics-AdaptiveGroups</th> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's an inconsistency in the formatting between the two instances of 'flowtrackedNetworkAnalytics-AdaptiveGroups'. In the first instance (line 22), it's plain text in a tag, but in the second instance (line 323), it's wrapped in tags. For consistency, both instances should use the same formatting.
This pull request addresses several spelling errors and formatting issues in the documentation, ensuring compliance with the style guide rules. Key changes include correcting misspellings, updating non-standard quotes, and fixing casing issues for specific terms.
schema-map.mdx)schema-map.mdx)schema-map.mdx)schema-map.mdx)schema-map.mdx)