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

FOR-469: Add character and word count components #171

Merged

Conversation

Rhyad-Zergane-HO
Copy link
Contributor

  • Update git ignore to ignore ds_store
  • Add hmpo-text-counter as base for new word and character counter; it implements gds character counter
  • Add hmpo-word-counter to count words, based on hmpo-text-counter
  • Add hmpo-character-counter to count characters, based on hmpo-text-counter
  • Add unit tests for each component
  • Add new global function to translate extra field keys
  • Add unit test for new global function

@Rhyad-Zergane-HO Rhyad-Zergane-HO force-pushed the FOR-469-create-new-character-counter-using-gds-component branch 2 times, most recently from 4694c34 to ba60064 Compare October 25, 2024 08:22
@Rhyad-Zergane-HO Rhyad-Zergane-HO force-pushed the FOR-469-create-new-character-counter-using-gds-component branch 2 times, most recently from 196125f to a076a4b Compare October 25, 2024 12:38
@Rhyad-Zergane-HO Rhyad-Zergane-HO force-pushed the FOR-469-create-new-character-counter-using-gds-component branch from a076a4b to 2886838 Compare October 25, 2024 12:49
Copy link

@matthewdoreilly matthewdoreilly 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. Just one spelling mistake

@Rhyad-Zergane-HO Rhyad-Zergane-HO force-pushed the FOR-469-create-new-character-counter-using-gds-component branch 3 times, most recently from 42c8498 to 282e41c Compare October 28, 2024 07:28
@Rhyad-Zergane-HO Rhyad-Zergane-HO force-pushed the FOR-469-create-new-character-counter-using-gds-component branch 2 times, most recently from a9a30ad to b4e387a Compare October 28, 2024 09:40
@Rhyad-Zergane-HO Rhyad-Zergane-HO force-pushed the FOR-469-create-new-character-counter-using-gds-component branch 3 times, most recently from 5141c16 to 285bb0e Compare October 28, 2024 10:22
Copy link
Contributor

@SamChatfield SamChatfield left a comment

Choose a reason for hiding this comment

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

LGTM

@Rhyad-Zergane-HO Rhyad-Zergane-HO force-pushed the FOR-469-create-new-character-counter-using-gds-component branch from 285bb0e to 3f0e748 Compare October 28, 2024 10:49
@Rhyad-Zergane-HO Rhyad-Zergane-HO force-pushed the FOR-469-create-new-character-counter-using-gds-component branch from 3f0e748 to c45c333 Compare October 28, 2024 10:50
@georgeeker georgeeker self-requested a review October 28, 2024 11:45
@BecioProton BecioProton merged commit 7d0259b into master Oct 28, 2024
2 checks passed
@BecioProton BecioProton deleted the FOR-469-create-new-character-counter-using-gds-component branch October 28, 2024 12:34
SamChatfield pushed a commit that referenced this pull request Oct 28, 2024
…r-using-gds-component

FOR-469: Add character and word count components
SamChatfield pushed a commit that referenced this pull request Oct 28, 2024
…r-using-gds-component

FOR-469: Add character and word count components
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.

7 participants