A setting should be added to allow users to hide messages from addresses not in their contacts list behind a 'show message' button. Related to T2418.
Details
Details
- Reviewers
bytesofman - Group Reviewers
Restricted Project
cd web/cashtab && npm start
in the browser:
observe that all messages are visible by default
navigate to settings tab and toggle 'show messages' off
receive a cashtab message from an address not in contact list
receive an encrypted message from an address not in contact list
receive a message from an address that IS in contact list
receive an encrypted message from an address that IS in the contact list
from the home tab:
observe that mesages from contacts are displayed
observe that the 'show message' button is displaying for messages that aren't from contacts
click the 'show message' button on both the encrypted and unencrypted message
observe that the messages are displayed, along with a warning that the sender is not a contact
note: changes are planned for this diff
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- hide-unknown-sender-msgs
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 18991 Build 37743: Build Diff cashtab-tests Build 37742: arc lint + arc unit
Event Timeline
Comment Actions
Build Bitcoin ABC Diffs / Diff Testing (cashtab-tests) failed.
Failed tests logs:
====== CashTab Unit Tests: Validation utils Recognizes a valid cashtab settings object ======
Error: expect(received).toBe(expected) // Object.is equality
Expected: true
Received: false
at Object.<anonymous> (/work/web/cashtab/src/utils/__tests__/validation.test.js:267:11)
at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
at new Promise (<anonymous>)
at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
at processTicksAndRejections (node:internal/process/task_queues:94:5)
at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:60:9)
at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)Each failure log is accessible here:
CashTab Unit Tests: Validation utils Recognizes a valid cashtab settings object
Comment Actions
Build Bitcoin ABC Diffs / Diff Testing (cashtab-tests) failed.
Failed tests logs:
====== CashTab Unit Tests: Validation utils Recognizes a valid cashtab settings object ======
Error: expect(received).toBe(expected) // Object.is equality
Expected: true
Received: false
at Object.<anonymous> (/work/web/cashtab/src/utils/__tests__/validation.test.js:267:11)
at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
at new Promise (<anonymous>)
at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
at processTicksAndRejections (node:internal/process/task_queues:94:5)
at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:60:9)
at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)
====== CashTab Unit Tests: Wallet with BCH balances and tokens ======
Error: expect(received).toMatchSnapshot()
Snapshot name: `Wallet with BCH balances and tokens 1`
- Snapshot - 1
+ Received + 1
@@ -489,11 +489,11 @@
<path
d="M832 464h-68V240c0-70.7-57.3-128-128-128H388c-70.7 0-128 57.3-128 128v224h-68c-17.7 0-32 14.3-32 32v384c0 17.7 14.3 32 32 32h640c17.7 0 32-14.3 32-32V496c0-17.7-14.3-32-32-32zM540 701v53c0 4.4-3.6 8-8 8h-40c-4.4 0-8-3.6-8-8v-53a48.01 48.01 0 1156 0zm152-237H332V240c0-30.9 25.1-56 56-56h248c30.9 0 56 25.1 56 56v224z"
/>
</svg>
</span>
- Show Msgs From Unkown Sender
+ Show Msgs From Unknown Sender
</div>
<button
aria-checked={true}
className="ant-switch ant-switch-small ant-switch-checked"
onClick={[Function]}
at Object.<anonymous> (/work/web/cashtab/src/components/Configure/__tests__/Configure.test.js:43:18)
at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
at new Promise (<anonymous>)
at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
at processTicksAndRejections (node:internal/process/task_queues:94:5)
at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)
====== CashTab Unit Tests: Without wallet defined ======
Error: expect(received).toMatchSnapshot()
Snapshot name: `Without wallet defined 1`
- Snapshot - 1
+ Received + 1
@@ -489,11 +489,11 @@
<path
d="M832 464h-68V240c0-70.7-57.3-128-128-128H388c-70.7 0-128 57.3-128 128v224h-68c-17.7 0-32 14.3-32 32v384c0 17.7 14.3 32 32 32h640c17.7 0 32-14.3 32-32V496c0-17.7-14.3-32-32-32zM540 701v53c0 4.4-3.6 8-8 8h-40c-4.4 0-8-3.6-8-8v-53a48.01 48.01 0 1156 0zm152-237H332V240c0-30.9 25.1-56 56-56h248c30.9 0 56 25.1 56 56v224z"
/>
</svg>
</span>
- Show Msgs From Unkown Sender
+ Show Msgs From Unknown Sender
</div>
<button
aria-checked={true}
className="ant-switch ant-switch-small ant-switch-checked"
onClick={[Function]}
at Object.<anonymous> (/work/web/cashtab/src/components/Configure/__tests__/Configure.test.js:63:18)
at Promise.then.completed (/work/web/cashtab/node_modules/jest-circus/build/utils.js:391:28)
at new Promise (<anonymous>)
at callAsyncCircusFn (/work/web/cashtab/node_modules/jest-circus/build/utils.js:316:10)
at _callCircusTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:218:40)
at processTicksAndRejections (node:internal/process/task_queues:94:5)
at _runTest (/work/web/cashtab/node_modules/jest-circus/build/run.js:155:3)
at _runTestsForDescribeBlock (/work/web/cashtab/node_modules/jest-circus/build/run.js:66:9)
at run (/work/web/cashtab/node_modules/jest-circus/build/run.js:25:3)
at runAndTransformResultsToJestFormat (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapterInit.js:170:21)
at jestAdapter (/work/web/cashtab/node_modules/jest-circus/build/legacy-code-todo-rewrite/jestAdapter.js:82:19)
at runTestInternal (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:389:16)
at runTest (/work/web/cashtab/node_modules/jest-runner/build/runTest.js:475:34)
at Object.worker (/work/web/cashtab/node_modules/jest-runner/build/testWorker.js:133:12)Each failure log is accessible here:
CashTab Unit Tests: Validation utils Recognizes a valid cashtab settings object
CashTab Unit Tests: Wallet with BCH balances and tokens
CashTab Unit Tests: Without wallet defined
Comment Actions
- Default behavior should be not to change a setting, so this logic should be reversed
e.g. the setting option should be "Hide messages from unknown senders" and it should be off by default.
- The 'reply to message' button should not be visible if the message is not shown
- Clicking "show message" should have an elegant effect to expand and reveal the msg, not content jumping. May be able to do this with only css transition properties. Don't use a collapse component. If it won't work with pure css, then create a task to do this separately.
- Create a task to get rid of all these classNames in the <GeneralSettingsItem> components + replace with styled components
Comment Actions
Default behaviour is changed, setting is now 'Hide messages from unkown senders', and its default state is off. The Reply to Message button is not shown if the message is hidden. Clicking 'show message' includes an transition effect on max-height. Updated snapshots and rebased to master.
Comment Actions
This seems to impact the ability to receive encrypted msgs.
user story
arc patch 11440
npm start
Cashtab loads showing an encrypted msg received yesterday as in the screenshot
Send another encrypted msg to the tested wallet, issue repeats
I didn't change the showMessages toggle so must be something in the logic changes to Tx.js
Comment Actions
This seems to impact the ability to receive encrypted msgs.
user story
arc patch 11440
npm startCashtab loads showing an encrypted msg received yesterday as in the screenshot
Send another encrypted msg to the tested wallet, issue repeats
I didn't change the showMessages toggle so must be something in the logic changes to Tx.js
I'm seeing this in prod today too actually -- so it's a separate issue, disregard.
Comment Actions
This seems to impact the ability to receive encrypted msgs.
user story
arc patch 11440
npm startCashtab loads showing an encrypted msg received yesterday as in the screenshot
Send another encrypted msg to the tested wallet, issue repeats
I didn't change the showMessages toggle so must be something in the logic changes to Tx.js
In testing, this also seems related to T2459, so does not need its own task
Comment Actions
- User should be able to see that this message is hidden because it is from an unknown sender
- Add padding between "Hide Message" and "Reply to Message" buttons
- while we are at it, let's get a task in to change the text of "Reply to Message" button to just "Reply"
- and while we're at that, in this diff please change text "Show Message" / "Hide Message" to "Show" / "Hide"
- It looks like this diff is hiding the "Reply to message" button from messages that do come from known senders