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

chore: add a user feedback in case of an error #171

Merged
merged 1 commit into from Jun 11, 2023

Conversation

a0m0rajab
Copy link
Contributor

@a0m0rajab a0m0rajab commented Jun 6, 2023

This PR adds a toast promise to the generateAiDescription function in posthighlight.tsx. The toast promise displays a loading message wThis feedback would show the user a message if something went wrong while generating AI for post to highlight

This adds an error message feedback for user when they face an issue in the AI generation on the post to highight popup

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Description

This adds an error message feedback for user when they face an issue in the AI generation on the post to highight popup

Related Tickets & Documents

#168

Mobile & Desktop Screenshots/Recordings

While Generating:
image

Successful message:
image

Error Message:
image

Added tests?

  • πŸ‘ yes
  • πŸ™… no, because they aren't needed
  • πŸ™‹ no, because I need help

Added to documentation?

  • πŸ“œ README.md
  • πŸ““ docs.opensauced.pizza
  • πŸ• dev.to/opensauced
  • πŸ“• storybook
  • πŸ™… no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

This feedback would show the user a message if something went wrong while generating
AI for post to highlight
@bdougie bdougie changed the title chore: add a user feedback in case if an error chore: add a user feedback in case of an error Jun 6, 2023
@bdougie
Copy link
Member

bdougie commented Jun 6, 2023

Can you add a screenshot @a0m0rajab?

@a0m0rajab
Copy link
Contributor Author

@bdougie added.

@diivi
Copy link
Collaborator

diivi commented Jun 6, 2023

where does "Configuration file is empty" come from? Doesn't seem like a helpful error.

@a0m0rajab
Copy link
Contributor Author

@diivi from this line:

throw new Error("Configuration file is empty!");

I manually through it to get the error screenshot.

@diivi
Copy link
Collaborator

diivi commented Jun 10, 2023

throw new Error("Configuration file is empty!");

Not so sure about "file", but it was there before too, so okay!

@a0m0rajab
Copy link
Contributor Author

a0m0rajab commented Jun 10, 2023 via email

Copy link
Member

@bdougie bdougie left a comment

Choose a reason for hiding this comment

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

Looks like there are some unrelated warning messages for unused variables. Everything else looks good.

currently failing on #173 and can be fixed there.

@bdougie bdougie merged commit 98d0681 into open-sauced:beta Jun 11, 2023
24 checks passed
@a0m0rajab a0m0rajab deleted the chore-add-feedback-ai branch June 11, 2023 20:01
@github-actions
Copy link

πŸŽ‰ This PR is included in version 1.6.0-beta.1 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

@github-actions
Copy link

πŸŽ‰ This PR is included in version 1.6.0 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants