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

Update ai-text-to-calendar extension #16186

Merged
merged 4 commits into from
Jan 15, 2025

Conversation

ViGeng
Copy link
Contributor

@ViGeng ViGeng commented Jan 6, 2025

Description

  • 🐞 Bug fix - preference API call and author id
  • πŸ‘¨β€πŸ³ Customizable service - fill in your own LLM service endpoint and model name
  • πŸ’¬ Multiple Language Support - Set your preferred language for events
  • 🍧 Minor prompt adjustment

Screencast

image

Checklist

- fix lint
- custom service
- [add] multiple language and custom endpoint support
- Initial commit
@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: ai-text-to-calendar Issues related to the ai-text-to-calendar extension labels Jan 6, 2025
@raycastbot
Copy link
Collaborator

raycastbot commented Jan 6, 2025

Thank you for your contribution! πŸŽ‰

πŸ”” @izm51 you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

You can expect an initial review within five business days.

- fix lint
- minor change
Copy link
Contributor

@izm51 izm51 left a comment

Choose a reason for hiding this comment

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

Thanks for the update!
I’ve left some comments, but overall, it looks good to me.

extensions/ai-text-to-calendar/README.md Outdated Show resolved Hide resolved
extensions/ai-text-to-calendar/README.md Outdated Show resolved Hide resolved
extensions/ai-text-to-calendar/src/ai-text-to-calendar.ts Outdated Show resolved Hide resolved
- fix lint
- fix url bugs
- update doc
@ViGeng
Copy link
Contributor Author

ViGeng commented Jan 13, 2025

Thanks for the review and suggestions.
I have updated a bit.
And fix the link issue: it returns URL with blanks.
I have fixed it and tested. Please also pay attention to it when you review and test it. :)

@pernielsentikaer
Copy link
Collaborator

Do you want to test it again @izm51

@pernielsentikaer pernielsentikaer self-assigned this Jan 14, 2025
@izm51
Copy link
Contributor

izm51 commented Jan 14, 2025

LGTM πŸ‘

Copy link
Collaborator

@pernielsentikaer pernielsentikaer left a comment

Choose a reason for hiding this comment

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

Hi πŸ‘‹

Looks good to me, approved πŸ”₯

@raycastbot raycastbot merged commit b64c80a into raycast:main Jan 15, 2025
2 checks passed
Copy link
Contributor

Published to the Raycast Store:
https://raycast.com/izm51/ai-text-to-calendar

@raycastbot
Copy link
Collaborator

πŸŽ‰ πŸŽ‰ πŸŽ‰

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

@ViGeng ViGeng deleted the ext/ai-text-to-calendar branch January 15, 2025 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension: ai-text-to-calendar Issues related to the ai-text-to-calendar extension extension fix / improvement Label for PRs with extension's fix improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants