chore: adjust database custom prompt ui#7902
Conversation
Reviewer's GuideThis pull request refines the user interface for selecting a database for custom AI prompts. It achieves this by introducing a new onboarding flow using the File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
🥷 Ninja i18n – 🛎️ Translations need to be updatedProject
|
| lint rule | new reports | level | link |
|---|---|---|---|
| Missing translation | 174 | warning | contribute (via Fink 🐦) |
There was a problem hiding this comment.
Hey @richardshiue - I've reviewed your changes - here's some feedback:
- The
AiPromptDatabaseSelectorButtonnow fetchesViewPBindependently; consider if this data fetching should be managed byAiPromptSelectorCubitto centralize view-related state. - Since the backend now skips generating prompts from rows with empty content, ensure the UI clearly communicates to the user if a selected database yields no custom prompts due to this filtering.
- The
AiPromptSelectorCubit's behavior forselectDatabasehas changed when custom prompts are empty (no longer defaulting to "Featured"); confirm this new user flow is intended.
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
frontend/appflowy_flutter/lib/ai/widgets/ai_prompt_modal/ai_prompt_database_selector.dart
Show resolved
Hide resolved
1030dad to
ece4d2c
Compare
Feature Preview
PR Checklist
Summary by Sourcery
Refactor and improve the custom prompt database selector UI in the AI prompt modal, enhancing user experience and adding more flexible database selection functionality.
Bug Fixes:
Enhancements:
Chores: