24423 Added conditional dashboard redirect #703
Conversation
- on start, redirect to new dashboard if FF is True
severinbeauvais
requested review from
hfekete,
ketaki-deodhar and
JazzarKarim
|
| navigate(dashboardUrl) | ||
| return | ||
| } | ||
|
|
There was a problem hiding this comment.
I had hoped to add this code to main.ts -- before Vue was started -- but for FF targeting, the LD user first had to be updated (line 443, above).
This code redirects to the new business dashboard if the FF is True. This means we don't have to update Namerequest UI or Auth Web (My Business Registry) or anywhere else to redirect conditionally.
Note that $route.fullPath includes any URL parameters (such as Filing ID and Account).
If you want to test this, contact me with your IDIR and I'll update LaunchDarkly to target you only.
There was a problem hiding this comment.
Sample redirect URL when :
|
/gcbrun |
|
Temporary Url for review: https://business-filings-dev--pr-703-athj4a1p.web.app SB says, try this: https://business-filings-dev--pr-703-athj4a1p.web.app/BC0870895?param1=severin¶m2=123 It should redirect only if the FF is True for you. |
|
got to this a bit late but looks good |
Issue #: bcgov/entity#24423
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the business-filings-ui license (Apache 2.0).