Add unique constraint support to Collection UI#79
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Agent-Logs-Url: https://github.com/yash-pouranik/urBackend/sessions/4054dd76-63d4-4b96-85cd-44c1a8594b91 Co-authored-by: yash-pouranik <172860064+yash-pouranik@users.noreply.github.com>
Agent-Logs-Url: https://github.com/yash-pouranik/urBackend/sessions/4054dd76-63d4-4b96-85cd-44c1a8594b91 Co-authored-by: yash-pouranik <172860064+yash-pouranik@users.noreply.github.com>
unique constraint support to Collection UI
There was a problem hiding this comment.
Pull request overview
Adds UI support in the web dashboard schema builder to mark top-level primitive fields as unique, so the existing backend unique-index creation can be configured during collection creation. Also cleans up tracked build output by ignoring dist and removing committed dist artifacts.
Changes:
- Add a UNIQ column and a
uniquecheckbox for eligible fields (depth 1 + primitive types) inCreateCollection.jsx. - Default new fields to
unique: falseand clearuniquewhen switching a field toObject/Array/Ref. - Ignore and remove
apps/web-dashboard/distbuild artifacts from the repo.
Reviewed changes
Copilot reviewed 2 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/web-dashboard/src/pages/CreateCollection.jsx | Adds unique field support to the schema builder UI and payload. |
| apps/web-dashboard/.gitignore | Ignores dist to prevent build output from being tracked. |
| apps/web-dashboard/dist/vite.svg | Removes committed build artifact. |
| apps/web-dashboard/dist/index.html | Removes committed build artifact. |
| apps/web-dashboard/dist/assets/index-0T9gOOFg.css | Removes committed build artifact. |
| apps/web-dashboard/dist/logo.png | Removes committed build artifact (binary). |
| apps/web-dashboard/dist/LOGO_SQ.png | Removes committed build artifact (binary). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot apply changes based on the comments in this thread |
Agent-Logs-Url: https://github.com/yash-pouranik/urBackend/sessions/45ff42a9-22a8-472e-af6c-e9ece3b747a0 Co-authored-by: yash-pouranik <172860064+yash-pouranik@users.noreply.github.com>
Agent-Logs-Url: https://github.com/yash-pouranik/urBackend/sessions/45ff42a9-22a8-472e-af6c-e9ece3b747a0 Co-authored-by: yash-pouranik <172860064+yash-pouranik@users.noreply.github.com>
|
@coderabbitai ready to merge? |
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain🏁 Script executed: Length of output: 8809 🏁 Script executed: Length of output: 1887
Here's a quick status summary:
The only remaining item is that the two inline Copilot review threads (accessibility label + column alignment) are still marked as unresolved, which is causing the |
aria-labelto the unique checkbox for accessibilityaria-hidden="true"to the empty spacer div