feat: improve create-project flow with clear project type split (#12)#96
feat: improve create-project flow with clear project type split (#12)#96CHY9213 wants to merge 1 commit into
Conversation
- Replace 6-category project types with two clear options: New Project / Bug Fix - Large two-column tiles with improved typography - Conditional form fields: attach-repo section shows only for Bug Fix - Dynamic placeholders based on selected project type - Project type included in the submission payload - New CSS for split layout and bigger select tiles Closes mergeos-bounties#12
QA Verification Report — PR #96Target PR: #96 (feat: improve create-project flow with clear project type split #12) CI StatusAll 5 checks pass: Backend build and test ✅ | Web build (admin) ✅ | Web build (frontend) ✅ | Web build (scan) ✅ | Secret scan ✅ Code Review
Evidence Check
Recommendation✅ Approve with suggestions — Good UX simplification. Suggest: (1) Confirm "Attach repository required" is intentional for Bug Fix; (2) Verify no backend/analytics dependencies on removed project types; (3) Add screenshots showing the new 2-option project type split. Verification per issue #64 — 300 MRG QA bounty |
|
Thanks for the PR. For bounty review, please add verification evidence in this PR before final review:
Evidence can be attached in a PR comment; images in comments count. If this PR has the |
TUPM96
left a comment
There was a problem hiding this comment.
Not ready for bounty acceptance yet.
This still appears to leave the repo-import path on the old Repo Issue Fix value while the UI introduces New Project / Bug Fix. That creates inconsistent state after a successful import and undercuts the main #12 requirement.
Please update repo import and all submitted/evaluation payload fields to use the new taxonomy consistently, then attach runtime evidence for both flows.
|
Thanks for the contribution and for moving this bounty forward. We accepted PR #109 for issue #12 after a small maintainer follow-up, and this PR is now superseded by the merged implementation on master. Closing to keep the queue clean. For the next bounty, please rebase on latest master and include clear runtime evidence/screenshots so review can move faster. |
Verification Report — PR #96Target PR: #96 Commands: git fetch+checkout ✅ | git diff master ✅ | Code review ✅Final Verdict: ✅ APPROVEPayout: cerouber88@gmail.com (PayPal) |
Summary
Replaced the 6-category project type grid (Web Development, Mobile, AI/ML, etc.) with two clear options:
Changes
frontend/src/App.vue
projectTypeOptionsto two objects: 'New Project' and 'Bug Fix'projectTypeto submission payloadfrontend/src/styles.css
.project-type-split(2-column grid, max-width 520px).select-tile-big(larger tiles, bigger text).new-project-hintstylesEvidence
(Add screenshots here after testing)
Closes #12