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

Integration of Spellcheck Feature for Resume #31

Merged
merged 2 commits into from
Sep 30, 2024
Merged

Conversation

ferdeleong
Copy link
Contributor

Fetch resume data (experience, education, and skills), and calls spell-check endpoint to check for spelling errors, and returns suggestions for corrections.

Added a new button labeled "Check Spelling" that triggers the spellcheck feature. When clicked, it fetches resume data (if not already loaded), sends it to the spellcheck API, and displays the spelling correction suggestions.

The spellcheck results are displayed in a list, showing the original text and the suggested corrections.

Closes #7

Fetch data and applies spellcheck to it
@ferdeleong ferdeleong self-assigned this Sep 30, 2024
{spellcheckResults
.filter((result) => result.after.length > 0)
.map((result, index) => (
<div key={index}>
Copy link
Contributor

Choose a reason for hiding this comment

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

It works perfectly, but maybe make this as a table so it is easier to follow.

Copy link
Contributor

@yalsaffar yalsaffar left a comment

Choose a reason for hiding this comment

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

LGTM!

@ferdeleong ferdeleong merged commit 6daa705 into main Sep 30, 2024
2 checks passed
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.

Spellcheck Functionality
2 participants