Skip to content

[engine][Target: wip-tenants] MultiTenant featured entities minds#4389

Ben requested to merge feat/featured-entities-4389 into master

Ticket(s) / Related Merge Requests

BLOCKED BY !1495 (merged)

Summary 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

delete-removes-user

Adding a user and pagination

feat-user-adding-a-user-and-pagination

User already added

add-existing-user

No user state

image

Auto-complete

image

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

Merge request reports