Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update URLs with account id #10880

Open
2 of 13 tasks
shabeeb-aot opened this issue Jan 28, 2022 · 5 comments
Open
2 of 13 tasks

Update URLs with account id #10880

shabeeb-aot opened this issue Jan 28, 2022 · 5 comments
Labels
Assets Dependency To highlight the dependency(intra team, external, third parties) ENTITY Business Team Priority1 STORY

Comments

@shabeeb-aot
Copy link
Contributor

shabeeb-aot commented Jan 28, 2022

See also Scott's spreadsheet.

Description

As a part of ticket #6151, the latest common components (v 2.4.38) can accept accountid as an URL parameter and set as current account in session storage. Some changes are needed in all page where a user can navigate between applications.

  1. Breadcrumbs
  2. Landing page links to application (may be handled by SbcHeader)
  3. Dashboard
  4. Navigation between apps, e.g; navigating to create or edit business from filings ui

Apps to update:

Expected Behavoir - Acceptance Criteria

  • GIVEN I am a user with multiple accounts
  • WHEN I select another account
  • THEN the current app, and apps that open via links, use the new account.

For example:

  • if in Auth Web, you stay in Auth Web
  • if on a BC Registries "marketing" page, redirect to the same "marketing" page for the selected account
  • if in a BC Registries application, redirect to dashboard for login type and account
    • what about staff - multiple accounts?
  • if on Name Request, redirect to Name Request for the selected account

Contact person: Faiz or Severin

@shabeeb-aot shabeeb-aot added the Dependency To highlight the dependency(intra team, external, third parties) label Jan 28, 2022
@severinbeauvais
Copy link
Collaborator

@Kaineatthelab @lmcclung What is the priority of this ticket?

@severinbeauvais
Copy link
Collaborator

Changed to higher priority than login/logout issues per PO sync today.

@jyoti3286
Copy link
Contributor

@severinbeauvais @shabeeb-aot - Do we need changes on PPR UI as well?

CC: @mstanton1 @Kaineatthelab

@severinbeauvais
Copy link
Collaborator

Do we need changes on PPR UI as well?

Yes. The latest sbc-common-components handle "receiving" the account id and setting it in the Sbc Header, but any links from PPR to other BCROS apps should include the account id param so the target app can be in sync.

@mstanton1
Copy link
Collaborator

@severinbeauvais I've created Zenhub 11068 for our team to tackle next sprint (ideally). I have limited understanding of what is needed. Can you fill in a few basic details in the ticket for my team?

CC @Kaineatthelab @jyoti3286

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Assets Dependency To highlight the dependency(intra team, external, third parties) ENTITY Business Team Priority1 STORY
Projects
None yet
Development

No branches or pull requests

7 participants