16637 Don't show numbered flow for CP + variable Incorporate Now button label

[ketaki-deodhar]

Issue #: /bcgov/entity#16637

Description of changes:

• For Coop don't show numbered.
• Re-use the existing feature flag for link to COLIN (until BC/CCC/ULC are released).
• Verify remaining flow.
• Trying to add/fix unit tests with no luck yet. Still looking into it
• Use FF to determine whether to redirect to COLIN or Continue In Now

Temp url: https://namerequest-dev--pr-639-c9rqzr4l.web.app/

Some screenshots showing the Continuation In flow:

COOP:

Limited company Named:

Approved NR Limited company Named:

Limited company Numbered: Will always show link to Colin as Continuation In applications are not yet implemented

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namerequest license (Apache 2.0).

[ketaki-deodhar]

/gcbrun

[pwei1018]

Temporary Url for review: https://namerequest-dev--pr-639-c9rqzr4l.web.app

[severinbeauvais]

When the NR is approved, what does the Existing NR page look like for your Continuation In examples above?

[ketaki-deodhar]

@severinbeauvais this is how approved NR looks. It does not show 'Incorporate Now' button.

Approved NR Limited company Named:

supported entity check:

[severinbeauvais]

Thanks for the approved Continuation In NR. Should it should a link to COLIN?

[ketaki-deodhar]

Thanks for the approved Continuation In NR. Should it should a link to COLIN?

I looked in the code and there aren't any instances we show COLIN link in this component but definitely can be added. Should I check with Yui/Janis?

[severinbeauvais]

Thanks for the approved Continuation In NR. Should it should a link to COLIN?

I looked in the code and there aren't any instances we show COLIN link in this component but definitely can be added. Should I check with Yui/Janis?

We just decided this morning: no button to COLIN. (Same thing we have now.)

I will add related comments to the ticket regarding this

[ketaki-deodhar]

/gcbrun

[pwei1018]

Temporary Url for review: https://namerequest-dev--pr-639-c9rqzr4l.web.app

[ketaki-deodhar]

LGTM but please get another review or 2 before merging. (You could ask Travis to add their new dev to this PR.)

as a reviewer?

[severinbeauvais]

LGTM but please get another review or 2 before merging. (You could ask Travis to add their new dev to this PR.)

as a reviewer?

Yes, but I'm mixed up. This is Namerequest. That other dev is working in Auth Web.

I'll ask Eve. She likes Namerequest I think 😄

[ketaki-deodhar]

~@seeker25 can you please add your new dev to this PR?~

@seeker25 Disregard ^^

[seeker25]

LGTM