[front] Live supermind replies minds#3426
Ticket(s) / Related Merge Requests
- Issue Card: minds#3426 (closed)
- Engine MR: engine!1419 (merged)
Summary of Changes
Adds live supermind reply type for Superminds that does not require a post to accept.
Testing Considerations
Note modal padding varies from designs as we're reusing an existing modal - I confirmed that this is fine with @mwrobs.
Creation
-
After selecting a "Live" Supermind and clicking to save, you see the appropriate modal. Canceling out of the modal does not dismiss Supermind popup, clicking to confirm the modal action does. -
In the Supermind popup, if you have Live selected you cannot select the checkbox to require a Twitter reply; it is greyed out. -
If you have ALREADY selected to require a Twitter reply, and select a live type, the checkbox is unchecked automatically (confirmed this behavior with @markryansallee - deviation from issue card). -
You can create the Live Supermind.
Accepting
-
In the recipient's console, it shows Live in the requirements section. -
On clicking decline, it is declined. -
On clicking to accept, a modal is shown explaining that no post is going to be created, and you can accept, being credited with the funds. -
After accepting, the console shows the Supermind in an accepted state - it does NOT pop it, this is by design of the console - same on decline.
Other
-
Other Superminds can be made for non-live types.
Deployment Considerations
Standard deployment.
Regression Scope
Supermind creation, accepting and console flows.
Platform Affected (web, mobile, etc)
Web.
Developer Testing Completed
Manual, unit (karma/jasmine).
Screenshots / Screen Recording
Creating
simplescreenrecorder-2023-08-02_16.00.57
Twitter vs Live behavior
simplescreenrecorder-2023-08-02_15.59.49
Modal Cancel and Confirm behavior
simplescreenrecorder-2023-08-02_15.59.06
Decline and Accept
simplescreenrecorder-2023-08-02_16.02.01
Does this impact
-
Localization -
Dark/light mode -
Guest mode
Definition of Done Checklist
-
The Acceptance Criteria has been met -
Code is tested: Testing includes unit/spec, E2E/automated and manual testing -
Merge requests description has been filled out
Edited by Ben