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

Feat/enable variable editing in prompts #2

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tscburak
Copy link

@tscburak tscburak commented Feb 23, 2025

Description

  • Added functionality to allow users to edit variables in prompts when they contain variables. Implemented in prompt.chat.
  • Fixed an execution issue in storage.ts due to local storage access limitations in Chrome. Open for feedback if this change is unnecessary.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Performance improvement
  • Code refactor
  • Other (please describe):

Testing

The "Act as DevOps Engineer" prompt contains three editable variables. The title variable was modified and successfully saved to local storage for future use.

Screenshots (if applicable)

image
image

…ject in storage.ts, which is causing issues in the PromptsProvider component.
@tscburak tscburak changed the title feat: enable variable editing in prompts Feat/enable variable editing in prompts Feb 23, 2025
@fatihsolhan
Copy link
Owner

Hey @tscburak, thanks for the PR! I'm a bit busy this week, but I'll definitely take a good look at this by the end of the week. Really appreciate you contributing! 🙌

@tscburak
Copy link
Author

Hey @tscburak, thanks for the PR! I'm a bit busy this week, but I'll definitely take a good look at this by the end of the week. Really appreciate you contributing! 🙌

You're very welcome! Whenever you get the chance to review it is totally fine. In the meantime, I'll work on another PR to save prompts to local storage and aim to have it ready by the end of the week. That way, you can review both together at your convenience. Thanks again! Looking forward to your feedback.

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.

2 participants