Skip to content

feat: product tags administration#249

Open
alukach wants to merge 2 commits intomainfrom
feat/product-tags-admin
Open

feat: product tags administration#249
alukach wants to merge 2 commits intomainfrom
feat/product-tags-admin

Conversation

@alukach
Copy link
Copy Markdown
Contributor

@alukach alukach commented Mar 25, 2026

Summary

  • Adds a "Tags" field to the product edit form (comma-separated input)
  • Updates the updateProduct server action to parse and save tags to product metadata

Closes #177

Changes

  • src/components/features/products/ProductCreationForm.tsx — added tags text field with comma-separated input, populated with existing tags in edit mode
  • src/lib/actions/products.tsupdateProduct now extracts tags from form data, splits on commas, trims whitespace, and saves to metadata.tags

Test plan

  • Verify tags field appears in product edit form
  • Verify existing tags are shown comma-separated when editing
  • Verify adding/removing tags persists after save
  • Verify empty tag input preserves existing tags
  • Verify tags display correctly on product pages after editing

🤖 Generated with Claude Code

alukach and others added 2 commits March 24, 2026 20:37
Add a tags field to the product edit form that accepts comma-separated
values. Tags are parsed, trimmed, and saved to the product metadata.

Closes #177

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
source-cooperative Ready Ready Preview, Comment Mar 25, 2026 4:28am

Request Review

@PowerChell
Copy link
Copy Markdown
Contributor

PowerChell commented Mar 25, 2026

My comment might not be for the admin side of tags... more user side! So might not be the right place to leave these comments !

Some small issues & ideas for the future:

  1. Suggested tag for when user types in something incomplete or misspelled. E.g. if the user wrote "sentinel" and did not define the number, suggesting all the "sentinel-" tags we have in the system. Or if someone misspells something or it is incomplete (e.g. geoparque or geoparquett are some examples I was playing with).
Screenshot 2026-03-25 at 9 28 45 AM
  1. Having a way for a user to see all tags possible and filter based on selecting existing tags instead of typing them in themselves
  2. A small issue, but the search bar has an autosearch functionality that I find is a bit fast. If I stop to think for a second, it searches for me when I am not done typing, and then says nothing is there. And then once it has searched, you have to re-click on the box to get the text back. I could see this being annoying if (1) someone was a slower typer (2) was just thinking of what they wanted to search, and was just pondering, and then it autocompletes for them before they are done. Maybe a small edge case but just paying with it, I noticed it, so it can't be THAT unusual. Literally 1-2 more seconds before it autocompletes searches for you could be nice / maybe these suggestions above would make this a non-issue.
  3. We are already having signs of duplicate tags that are slightly different from each other. We should be monitoring for this during the tag creation portion of the process.
Screenshot 2026-03-25 at 9 35 16 AM

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Proposed Feature] Product Tags administration

2 participants