[engine][Target: wip-tenants] MultiTenant featured entities minds#4389
Ticket(s) / Related Merge Requests
- Issue card: minds#4389 (closed)
- Front MR: front!2468 (merged)
!1495 (merged)
BLOCKED BYSummary of Changes
Testing Considerations
Notes
- The URL you can test on is: https://c81e728d9d4c2f636f067f89cc14862c.bens-networks-2.oke.minds.io
- Log in as this tenant network admin (same as the other sandbox)
- Design deviations have been adjusted on the fly and approved by @chints98 - there has been a significant change to the originally planned UI.
- Actual behavior to auto-subscribe is NOT implemented in this card, this is JUST the UI.
- The auto-complete WILL return users NOT from this tenant that you cannot add - this is to be fixed in another issue card.
- The Group tab has been omitted at instruction from @markharding.
Checklist
-
With NO users you see the appropriate Add a featured user
text. -
On adding a user auto-complete works (barring the above note). -
On add the user is appended to the list - the change for this persists on reload. -
On clicking the X next to a user will remove them from the list - the change for this persists on reload. -
If you try to add a user that isn't found nothing should be added and you should see a warning message. (also if you add one that is NOT on this tenant). -
The list should paginate (every 12 users).
List of users I made you can try adding / removing:
- admin
- testacc
- user1
- user2
- user3
- user4
- user5
- user6
- user7
- user8
- user9
- user10
- user11
- user12
- user13
- user14
Deployment Considerations
Need to add the table from the mysql-provisioner
file.
Regression Scope
General tab of networks.
Platform Affected (web, mobile, etc)
Web / mobile web.
Developer Testing Completed
Manual, unit (karma/jasmine/phpspec).
Screenshots / Screen Recording
Removing a user
Adding a user and pagination
feat-user-adding-a-user-and-pagination
User already added
No user state
Auto-complete
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