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

hide name check result container when name is too short #657

Merged

Conversation

bolyachevets
Copy link
Collaborator

@bolyachevets bolyachevets commented Aug 24, 2023

*Issue #: /bcgov/entity/16998

*Description of changes:

Screenshot 2023-08-23 at 4 59 31 PM

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

@bolyachevets
Copy link
Collaborator Author

/gcbrun

@pwei1018
Copy link
Collaborator

Temporary Url for review: https://namerequest-dev--pr-657-8w8uwoxl.web.app

@severinbeauvais
Copy link
Collaborator

Could this fix be applied to the Way of Navigating feature branch instead? (Feature branch is planned for deployment in October.)

The feature branch is in active development at the moment and any changes to main branch may conflict, or at least will require some careful rebasing/refactoring. Therefore, I'd prefer to merge this PR into the feature branch if possible.

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.

From what I know, this won't conflict with changes in the feature branch made up to now.

@bolyachevets bolyachevets changed the base branch from main to feature-way-of-navigating August 24, 2023 18:43
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 but please update the app version!

@bolyachevets
Copy link
Collaborator Author

/gcbrun

@severinbeauvais severinbeauvais merged commit a0b77b1 into bcgov:feature-way-of-navigating Aug 24, 2023
5 checks passed
@pwei1018
Copy link
Collaborator

Temporary Url for review: https://namerequest-dev--pr-657-8w8uwoxl.web.app

@bolyachevets bolyachevets deleted the 16998-name-check branch August 24, 2023 20:02
Copy link
Collaborator

@ozamani9gh ozamani9gh left a comment

Choose a reason for hiding this comment

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

great

severinbeauvais pushed a commit to severinbeauvais/namerequest that referenced this pull request Aug 29, 2023
* hide name check when name is too short

* update version
JazzarKarim pushed a commit to JazzarKarim/namerequest that referenced this pull request Sep 8, 2023
* hide name check when name is too short

* update version
JazzarKarim pushed a commit that referenced this pull request Sep 11, 2023
* hide name check when name is too short

* update version
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