Skip to content

feat(pwa,data): wire InstallPrompt analytics and investor StaleDataBadge - #749

Merged
king-aj-the-first merged 1 commit into
OpenLedger-Foundation:mainfrom
Otaiki1:feat/stellar-wave-708-711
Aug 31, 2026
Merged

feat(pwa,data): wire InstallPrompt analytics and investor StaleDataBadge#749
king-aj-the-first merged 1 commit into
OpenLedger-Foundation:mainfrom
Otaiki1:feat/stellar-wave-708-711

Conversation

@Otaiki1

@Otaiki1 Otaiki1 commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Linked Issue

Closes #

What Changed

Testing Steps

Screenshots or Recordings

Add before/after screenshots for UI changes, especially invoice, SME dashboard, investor marketplace, position, and wallet states.

Checklist

  • Tests added or updated
  • Docs updated, if needed
  • Accessibility checked
  • Wallet and network behavior considered
  • SME, investor, invoice, or position terminology is accurate
  • No secrets, private keys, real wallet seed phrases, or live credentials added

Notes for Reviewers

Call out any migration steps, mocked data, incomplete contract wiring, or areas that need focused review.
Closes #708
Closes #712
Closes #713
Closes #711

- InstallPrompt now emits install_prompt_shown/accepted/dismissed via
  lib/installPromptAnalytics.ts (Closes OpenLedger-Foundation#708)
- Investor dashboard renders StaleDataBadge from the positions query
  dataUpdatedAt, matching the marketplace header (Closes OpenLedger-Foundation#711)
- Document the install prompt events in docs/analytics-events.md
- Add a unit test for the install prompt analytics helper

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@drips-wave

drips-wave Bot commented Aug 31, 2026

Copy link
Copy Markdown

@Otaiki1 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@king-aj-the-first
king-aj-the-first merged commit c8f6ba9 into OpenLedger-Foundation:main Aug 31, 2026
9 of 11 checks 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

2 participants