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(ui, api): prompt template export #6745

Merged
merged 9 commits into from
Aug 16, 2024
Merged

Conversation

maryhipp
Copy link
Collaborator

Summary

Adds option to download all prompt templates to a CSV

Related Issues / Discussions

QA Instructions

Merge Plan

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • Documentation added / updated (if applicable)

@github-actions github-actions bot added api python PRs that change python files services PRs that change app services frontend PRs that change frontend files labels Aug 15, 2024
Copy link
Collaborator

@psychedelicious psychedelicious left a comment

Choose a reason for hiding this comment

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

Made a few changes:

  • Save the file as prompt_templates.csv
  • Use buttons instead of menu for import/export
  • Show all template types when no matching templates, moving fallback inside the the list view
  • Focus the prompt template textarea after inserting placeholder

@psychedelicious psychedelicious enabled auto-merge (rebase) August 16, 2024 00:16
auto-merge was automatically disabled August 16, 2024 00:26

Rebase failed

@psychedelicious psychedelicious merged commit 713bd11 into main Aug 16, 2024
14 checks passed
@psychedelicious psychedelicious deleted the maryhipp/export-presets branch August 16, 2024 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api frontend PRs that change frontend files python PRs that change python files services PRs that change app services
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants