hicommonwealth / commonwealth

A platform for decentralized communities
https://commonwealth.im
GNU General Public License v3.0
67 stars 41 forks source link

Notifications Improvements v1 Test Plan #8175

Closed timolegros closed 2 weeks ago

timolegros commented 2 months ago

Test Plan

In-App Notifications

Thread Subscriptions

  1. Sign in as user A
  2. Create a thread and subscribe to it
  3. Sign in as user B
  4. Comment on the same thread
    • [x] User A should receive a notification

Mention Subscriptions

  1. As user B create another comment that mentions user A
  2. As user B create a thread that mentions user A
    • [x] User A should receive a mention notification and a comment notification
    • [x] User A should receive a mention notification

Comment Subscriptions

  1. As user B create a thread.
  2. As user A comment on the thread.
  3. As user B reply to the comment.
    • [x] User A should receive a reply notification

Snapshot Notifications

  1. As user A join the dydx community
  2. Subscribe to CommunityAlerts from the subscriptions settings page
  3. Execute pnpm emit-event snapshot
    • [x] User A should receive a snapshot notification

Chain Event Notifications

  1. Execute the following query on the relevant database:
    UPDATE "Communities"
    SET namespace_address = '0xD7f82204b1F47BFdE583a8d360986b31F22D3DAe'
    WHERE id = 'dydx';
  2. Run pnpm set-super-admin (requires having SUPER_ADMIN_EMAIL or SUPER_ADMIN_WALLET_ADDRESS set in .env). If executing against a remote app just append frick, frack or beta to the end of the command.
  3. Run pnpm emit-event chain-event a couple of times in quick succession. Wait 1 minute for the batch to complete in Knock (batch time is currently 1 minute in Knock development workspace but 1 day in Knock Frack workspace so temporarily modify it for testing if needed).
    • [x] User A should receive a community stake mint notification

Issues

timolegros commented 1 month ago

Main goal for tomorrow (July 18) is to fill out this ticket and test in-app notifications end to end + resolve any remaining issues.

burtonator commented 1 month ago

The preview doesn't exactly look good:

CleanShot 2024-07-25 at 14 36 38@2x

timolegros commented 1 month ago

The preview doesn't exactly look good:

CleanShot 2024-07-25 at 14 36 38@2x

Did you make changes to how we truncate or decode bodies? This wasn't like this before.

timolegros commented 1 month ago

All notifications work for me - I think this is ready for QA if #8636 is merged.

timolegros commented 1 month ago

Making a note that I need to update the test plan to include subscriptions testing.