Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

19041 Created constants (enum) for resource phrases #647

Merged
merged 2 commits into from
Feb 6, 2024

Conversation

severinbeauvais
Copy link
Collaborator

Issue #: bcgov/entity#19041

Description of changes:

  • app version = 5.8.9
  • created constants (enum) for common resource phrases
  • updated misc resource files to use constants

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

- created constants (enum) for common resource phrases
- updated misc resource files to use constants
@severinbeauvais
Copy link
Collaborator Author

All, what do you think about this change -- worth doing or too abstract?

@@ -0,0 +1,11 @@
/* eslint-disable max-len */
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I thought I'd have the strings all on one line (no breaks). Thoughts?

Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

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.

I like it 😃

Copy link
Contributor

@jamespaologarcia jamespaologarcia left a comment

Choose a reason for hiding this comment

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

Beautiful. LGTM

Copy link
Collaborator

@ketaki-deodhar ketaki-deodhar left a comment

Choose a reason for hiding this comment

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

Worth it! :)

@severinbeauvais severinbeauvais merged commit 479d1aa into bcgov:main Feb 6, 2024
4 of 5 checks passed
JazzarKarim pushed a commit to JazzarKarim/business-create-ui that referenced this pull request Feb 23, 2024
* - app version = 5.8.9
- created constants (enum) for common resource phrases
- updated misc resource files to use constants

* - changed OFFENSE -> OFFENCE

---------

Co-authored-by: Severin Beauvais <severin.beauvais@gov.bc.ca>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants