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

fix: client registration events are on eventStore #2093

Merged
merged 1 commit into from
Sep 27, 2022

Conversation

gardleopard
Copy link
Contributor

Client registration events are on eventStore and not on eventBus. This change makes us have sdk name and version metrics in unleash.

@vercel
Copy link

vercel bot commented Sep 26, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Ignored Deployments
Name Status Preview Updated
unleash ⬜️ Ignored (Inspect) Sep 26, 2022 at 1:38PM (UTC)
unleash-docs ⬜️ Ignored (Inspect) Sep 26, 2022 at 1:38PM (UTC)
unleash-monorepo-frontend ⬜️ Ignored (Inspect) Sep 26, 2022 at 1:38PM (UTC)

@github-actions
Copy link

github-actions bot commented Sep 26, 2022

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
88.58% (-2.79% 🔻)
7098/8013
🟡 Branches
78.91% (-0.64% 🔻)
1104/1399
🟢 Functions
82.34% (-3.89% 🔻)
2000/2429
🟢 Lines
88.9% (-2.4% 🔻)
6583/7405

⚠️ Details were not displayed: the report size has exceeded the limit.

Test suite run success

1170 tests passing in 194 suites.

Report generated by 🧪jest coverage report action from 97c567d

Client registration events are on eventStore and not on eventBus. This
change makes us have sdk name and version metrics in unleash.
Copy link
Contributor

@gastonfournier gastonfournier left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@chriswk chriswk left a comment

Choose a reason for hiding this comment

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

Nice catch, you're absolutely correct. :)

@gardleopard gardleopard merged commit 9aa1c7a into main Sep 27, 2022
@gardleopard gardleopard deleted the fix_client_registration_metrics branch September 27, 2022 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants