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

Bugfix/TS 496 Self assessment extraction word counts #1275

Merged

Conversation

HalcyonJAC
Copy link
Contributor

@HalcyonJAC HalcyonJAC commented Dec 17, 2024

What's included?

Issue: User Raised Issue BR_ADMIN_PR_000274 #496

  • Upgrade the jac-kit version to 4.1.53.
  • Use the function splitWords from jac-kit in the FormField component.

Who should test?

✅ Product owner
✅ Developers

How to test?

Example vacancy

Steps:

  1. Apply for the vacancy and go to the "Self assessment with competencies" section.

  2. Upload a finished self assessment and check if the word count is correct.

    For example, the word count for It’s something about and It's something about should be 3.

Risk - how likely is this to impact other areas?

🟢 No risk - this is a self-contained piece of work

Additional context

Demo:

Screen.Recording.2024-12-18.at.09.49.57.mov

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

@HalcyonJAC HalcyonJAC self-assigned this Dec 17, 2024
Copy link

github-actions bot commented Dec 18, 2024

Visit the preview URL for this PR (updated for commit 6a2d6f1):

https://jac-apply-develop--pr1275-bugfix-ts-496-self-a-whdn4fy1.web.app

(expires Sun, 02 Feb 2025 11:05:31 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: d5ac0a9c931657844a433a502135cf38aca21627

@nickaddy
Copy link
Contributor

@HalcyonJAC For me, they are counting differently:

image

@HalcyonJAC
Copy link
Contributor Author

@nickaddy It only works for and '. Do we need to include ` as well?

@nickaddy
Copy link
Contributor

@HalcyonJAC Yes, pls. I thought I had the same ones, I couldn't find another. On a PC, the key to the left of 1 at the top of the keyboard and on the button with @. Where is the other one?

@HalcyonJAC
Copy link
Contributor Author

@nickaddy Could you test it again, please?

@HalcyonJAC HalcyonJAC merged commit 43d7f2b into main Jan 3, 2025
4 checks passed
@HalcyonJAC HalcyonJAC deleted the bugfix/ts-496-self-assessment-extraction-word-counts branch January 3, 2025 11:06
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.

5 participants