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

18159 Fixed name input error message for non-xpro #729

Merged
merged 3 commits into from
Oct 16, 2023

Conversation

severinbeauvais
Copy link
Collaborator

@severinbeauvais severinbeauvais commented Oct 16, 2023

Issue #: bcgov/entity#18159

Description of changes:

  • app version = 5.2.10
  • fixed Name Input error message logic
  • BUG FIX: added missing corp type / entity type conversions
  • also fixed some GHA warnings (lint-type errors)

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

@severinbeauvais severinbeauvais self-assigned this Oct 16, 2023
@severinbeauvais severinbeauvais changed the title 18159 18159 Fixed name input error message for non-xpro Oct 16, 2023
@severinbeauvais
Copy link
Collaborator Author

Test Results

New named company:
image

New sole proprietorship:
image

New xpro in MRAS jurisdiction:
image

New xpro in non-MRAS jurisdiction:
image

- fixed Name Input error message logic
}
return ['Please enter a name for the business']
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is the same logic as for the label (lines 87-96 above).

I always wondered why this logic was different. The answer is that it was wrong 😆

Copy link
Collaborator

@JazzarKarim JazzarKarim left a comment

Choose a reason for hiding this comment

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

LGTM!

@severinbeauvais
Copy link
Collaborator Author

@ketaki-deodhar Please test a bunch of name change scenarios to verify that they all work. Thanks.

@bcgov bcgov deleted a comment from bcregistry-sre Oct 16, 2023
@severinbeauvais
Copy link
Collaborator Author

/gcbrun

@bcgov bcgov deleted a comment from bcregistry-sre Oct 16, 2023
@bcregistry-sre
Copy link
Collaborator

Temporary Url for review: https://namerequest-dev--pr-729-2j989mi5.web.app

Copy link
Collaborator

@leodube-aot leodube-aot left a comment

Choose a reason for hiding this comment

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

Looks good Sev

@severinbeauvais severinbeauvais merged commit bf44782 into bcgov:main Oct 16, 2023
5 checks passed
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