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

[DO NOT MERGE] Credential Engine launch #2415

Open
wants to merge 1,050 commits into
base: main
Choose a base branch
from
Open

[DO NOT MERGE] Credential Engine launch #2415

wants to merge 1,050 commits into from

Conversation

ChelseaKR
Copy link
Collaborator

No description provided.

@ChelseaKR ChelseaKR force-pushed the credential-engine branch from 196d379 to c609043 Compare May 10, 2024 22:25
@ChelseaKR ChelseaKR force-pushed the credential-engine branch from da6bc08 to efa049a Compare July 5, 2024 16:13
@ChelseaKR ChelseaKR changed the base branch from master to main July 27, 2024 04:19
ChelseaKR and others added 30 commits February 5, 2025 16:19
…cified text should be hidden if provider does not give that info"
…n smaller screens

Added `word-wrap: break-word;` and `overflow-wrap: break-word;` to `.fact-item a`
to prevent long URLs from overflowing and improve readability on mobile devices.
- Refactored `rankResults` function:
  - Introduced `COMMON_WORDS` set to filter out generic terms.
  - Improved token-based matching with weighted scores.
  - Enhanced fuzzy matching efficiency.
  - Set a minimum score threshold for relevance filtering.

- Improved query filtering:
  - Standardized tokenization for training names, descriptions, and providers.
  - Boosted ranking for strong matches and removed low-relevance results.

- Added additional logging for debugging and performance insights.

This refactor enhances search accuracy, reduces noise, and improves result ranking consistency.
…index

NJWE-2630 - TE details header z-index
- Updated `findTrainingsBy.ts` to return an empty array instead of throwing an error when no CE records are found.
- Added a test case in `findTrainingsBy.test.ts` to verify this behavior.
- Ensured the function fails gracefully and does not produce unhandled errors in logs.

Resolves NJWE-2639.
[NJWE-2664] ensure provider URL wraps correctly on smaller screens
…-handle-missing-records

fix(training): handle missing CE records gracefully
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.

3 participants