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

23332 - Enable request restoration NR for a limited restored company #786

Conversation

AimeeGao
Copy link
Collaborator

@AimeeGao AimeeGao commented Dec 17, 2024

Issue #: /bcgov/entity#23332

Description of changes:

  • Updated BusinessLookupFetch component to use the exact lookup for restoration name requests

    • Use the BusinessFetch component for restoration NRs
    • Support exact look up no matter whether the user is logged in or not, and whether the business state is Active or Historical
    • Maintain existing lookup logic for non-restoration requests
  • Added check for limited restoration name requests

    • Verify the business is in limited restoration state by checking restorationExpiryDate before allowing the restoration NR
    • Reject the request if the business is not eligible for restoration NRs

Screenshots:

The user is not logged in:

Test locally with a limited restoration business: BC0883802

  1. Select Reactivate a BC or Extraprovincial company, and put in the business identifier:
    image

  2. Click on the search button:
    image

Test locally without a limited restoration business: BC0883817

Select Reactivate a BC or Extraprovincial company, put in the business identifier, and click on the search button:
image

Test locally with an expired limited restoration business: BC0871585

Select Reactivate a BC or Extraprovincial company, put in the business identifier, and click on the search button:
image

The user is logged in:

Test locally with a limited restoration business: BC0883802

  1. Select Reactivate a BC or Extraprovincial company, and put in the business identifier:
    image

  2. Click on the search button:
    image

Test locally without a limited restoration business: BC0883817

Select Reactivate a BC or Extraprovincial company, put in the business identifier, and click on the search button:
image

Test locally with an expired limited restoration business: BC0871585

Select Reactivate a BC or Extraprovincial company, put in the business identifier, and click on the search button:
image

Name Request Result:

image
image

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).

@AimeeGao
Copy link
Collaborator Author

/gcbrun

@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://namerequest-dev--pr-786-fm9k04h2.web.app

@AimeeGao
Copy link
Collaborator Author

/gcbrun

@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://namerequest-dev--pr-786-fm9k04h2.web.app

Copy link
Collaborator

@severinbeauvais severinbeauvais left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

app/package.json Outdated Show resolved Hide resolved
@AimeeGao AimeeGao force-pushed the 23332-request-restoration-NR-for-limited-restored-company branch from 0c906b2 to 471fae3 Compare December 19, 2024 18:18
@AimeeGao
Copy link
Collaborator Author

/gcbrun

@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://namerequest-dev--pr-786-fm9k04h2.web.app

@EPortman EPortman self-requested a review December 19, 2024 18:23
Copy link
Collaborator

@EPortman EPortman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great

@AimeeGao AimeeGao merged commit f383d77 into bcgov:main Dec 19, 2024
9 of 10 checks passed
meawong pushed a commit to meawong/namerequest that referenced this pull request Jan 3, 2025
…cgov#786)

* Added check for limited restoration

* Added condition to use exact look up (BusinessFetch) component for restoration NRs
meawong added a commit that referenced this pull request Jan 3, 2025
* 25066-Update-allowable-BEN-conversions

* 25066-Lint fixes

* 25066-Use-namexservice-to-fetch-business

* Add logic for displaying name change button

* update patch version

* Add feature flag check

* lint fix

* 24205 - updates for name change and alteration (#784)

* 24205 - updates for name change and alteration ff

* 24205 - remove feature flags

* 24205 - remove feature flags

* 24205 - update version

* 24205 - remove unsused import

* 24205 - remove consitional chaining

* 24205 - update funstions

* 24205

* 24205

* 24205- update mounted() with async

* 24605 additional address field error when editing nr (#785)

* 23332 - Enable request restoration NR for a limited restored company (#786)

* Added check for limited restoration

* Added condition to use exact look up (BusinessFetch) component for restoration NRs

* 23332 - Fix request restoration nr for limited restored company (#787)

* fix: allow historical businesses to request restoration NR

* 24841 display error when create payment fails (#788)

* 25066-Update-allowable-BEN-conversions

* 25066-Lint fixes

* 25066-Use-namexservice-to-fetch-business

* 25066-Remove-unecessary-import

* 25066-Remove fetch business service that is unused

* 25066-Update patch version

---------

Co-authored-by: ketaki-deodhar <[email protected]>
Co-authored-by: EPortman <[email protected]>
Co-authored-by: Aimee <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants