Skip to content

Default to NOT posting installs/uninstalls to slack - #22

Merged
coladarci merged 1 commit into
mainfrom
feature/change-metrics-default-behavior
Aug 11, 2025
Merged

Default to NOT posting installs/uninstalls to slack#22
coladarci merged 1 commit into
mainfrom
feature/change-metrics-default-behavior

Conversation

@coladarci

Copy link
Copy Markdown
Collaborator

We wanted visibility into new app installs but we are learning that they are so spammy it's annoying. Behavior can be turned on for apps you want it, but either way you'll get a weekly summary which is "good enough" for most apps.

@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 0436e26cafa0493c3148d9f7c13e81e15b9eb9ae-PR-22

Details

  • 2 of 4 (50.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 27.5%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/peek_app_sdk/metrics.ex 0 2 0.0%
Totals Coverage Status
Change from base Build efffd5916f0e6f034b2c1d2bd6edd24c19ffb8bc: 0.0%
Covered Lines: 110
Relevant Lines: 400

💛 - Coveralls

@coladarci
coladarci merged commit c4eb74c into main Aug 11, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants