Closed camelPhonso closed 1 month ago
Name | Link |
---|---|
Latest commit | 2c7e49b5df7bfc2e3877fcd77d18e8f72403c068 |
Latest deploy log | https://app.netlify.com/sites/cool-creponne-3e1272/deploys/66e9400ea2dbbc000889a19f |
Deploy Preview | https://deploy-preview-332--cool-creponne-3e1272.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
@camelPhonso I cant see any dots and this test fails
this test fails
🤔 I had the same until I ran npx supabase db reset
and then it behaved normal.
Description
Closes #329
Bug fix for displaying a notification dot over any conversation card to a conversation with unread messages.
Extended tests to our messaging feature to avoid regression.
Files changed
supabase/migrations/20240906121523_**.sql
- new migration file recreating thefetch_user_conversations
function in our database to return the value ofhas_unread_messages
in each conversationConversationsList.tsx
- added useEffect to updatenotificationsList
local state when changes occur toallConversations
in the wider context.components/messaging/**.tsx
&&app/styles/messaging-styltes.css
- lifted some stylings and addedname
attributes to some elements for testability of the messaging functionalitymessaging.cy.js
,URLS.json
&&conversationsPage.js
- new testing suite for messaging featureUI changes
none - everything looks the same as before the bug was introduced
Changes to Documentation
none
Tests
New test suite verifies that: