Skip to content

Conversation

@omarmosid
Copy link
Contributor

@omarmosid omarmosid commented Dec 30, 2024

Summary

Adding docs for the new feature that allows exporting zaraz logs via logpush.

Screenshots (optional)

Documentation checklist

  • The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 30, 2024

Deploying cloudflare-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: a5ddaa9
Status: ✅  Deploy successful!
Preview URL: https://74f45844.cloudflare-docs-7ou.pages.dev
Branch Preview URL: https://omar-zaraz-logpush.cloudflare-docs-7ou.pages.dev

View logs

Zaraz offers privacy settings you can configure, such as:

* **Remove URL query parameters**: Removes all query parameters from URLs. For example, `https://example.com/?q=hello` becomes `https://example.com/`.
- **Remove URL query parameters**: Removes all query parameters from URLs. For example, `https://example.com/?q=hello` becomes `https://example.com/`.
Copy link
Member

Choose a reason for hiding this comment

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

let's remove this fomatting noise from the PR please - probably an update to editor settings needed

Copy link
Contributor

@hyperlint-ai hyperlint-ai bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 2 total issue(s) found.

omarmosid and others added 4 commits January 3, 2025 11:35
Co-authored-by: hyperlint-ai[bot] <154288675+hyperlint-ai[bot]@users.noreply.github.com>
@omarmosid omarmosid requested a review from jonnyparris January 3, 2025 15:06
Copy link
Member

@jonnyparris jonnyparris left a comment

Choose a reason for hiding this comment

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

Let's add a changelog entry too then it's ready to merge 👍🏾

@omarmosid
Copy link
Contributor Author

Let's add a changelog entry too then it's ready to merge 👍🏾

Added 🙏🏽

@jonnyparris jonnyparris merged commit 6d00c4d into production Feb 12, 2025
12 checks passed
@jonnyparris jonnyparris deleted the omar/Zaraz-logpush branch February 12, 2025 09:43
jonesphillip pushed a commit that referenced this pull request Feb 12, 2025
* ZRZ-1495 Add docs for logpush

* Apply suggestions from code review

Co-authored-by: Ruskin <[email protected]>

* Remove '-'

* Add back extra *

* Fix spacing

* Apply suggestions from code review

Co-authored-by: hyperlint-ai[bot] <154288675+hyperlint-ai[bot]@users.noreply.github.com>

* Add changelog entry for logpush

---------

Co-authored-by: Ruskin <[email protected]>
Co-authored-by: hyperlint-ai[bot] <154288675+hyperlint-ai[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants