-
Notifications
You must be signed in to change notification settings - Fork 0
First implementation #1
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
Conversation
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.
Pull Request Overview
This PR adds the initial implementation of a Woopra component for Edgee, using Woopra’s HTTP Tracking API to forward page, custom track, and user-identify events via GET querystrings.
- Introduces
WoopraPayloadTrackandWoopraPayloadIdentifyfor serializing event and user data - Implements
Guestmethods insrc/lib.rsto map Edgee events to Woopra GET requests - Adds component manifest, documentation, Makefile, CI workflows, and basic tests
Reviewed Changes
Copilot reviewed 10 out of 11 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| src/woopra_payload.rs | Payload structs and custom serde serializers for Woopra prefixes |
| src/lib.rs | Component logic for page/track/user events and corresponding tests |
| edgee-component.toml | Component manifest with build and settings configuration |
| README.md | Usage guide, quick start, and configuration options |
| Makefile | Local build, test, and coverage targets |
| Cargo.toml | Package metadata and dependencies |
| CONTRIBUTING.md | Contributor guidelines |
| .github/workflows/wasm-build-release.yml | Release workflow for building and uploading the WASM artifact |
| .github/workflows/check.yml | CI checks: formatting, lint, build, and tests |
| .github/CODEOWNERS | Codeowners configuration |
Comments suppressed due to low confidence (2)
.github/workflows/check.yml:67
- This check is looking for
clickhouse.wasm, but the component buildswoopra.wasm. Update the filename to match.
if [ ! -f "./clickhouse.wasm" ]; then
src/woopra_payload.rs:292
- This comment precedes
serialize_cs_prefixedbut mentionsce_. Update it to reflect thecs_prefix.
// Helper function to serialize HashMap with "ce_" prefix
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
I've tested this with: edgee component test --event-type track --settings project_name=demo.edgee.app --make-http-request
edgee component test --event-type page --settings project_name=demo.edgee.app --make-http-request
edgee component test --event-type user --settings project_name=demo.edgee.app --make-http-request
and it works great 👌 Note: the default test event uses a very old timestamp (from Feb 2025), so the event is ingested but doesn't show up in the real-time dashboard :) |
|
Also tested on staging on demo.edgee.team and it's working great 👌 |
Checklist
Description of Changes
Details: