monyi/greasysprocket-website
1
0
Fork 0
Code Issues 1 Pull requests 1 Projects Releases Packages Wiki Activity Actions

feat(feedback): selector de extensión con preselección por ?ext #3

Open
monyi wants to merge 0 commits from feat/feedback-ext-selector into main
pull from: feat/feedback-ext-selector
merge into: monyi:main
Conversation 0 Commits - Files changed -
monyi commented 2026-10-22 22:32:08 +02:00
Owner
Copy link

El parámetro ?ext existía pero no había forma de elegir extensión desde el formulario.

  • hidden -> select con 28 extensiones (nombre visible, id canónico)
  • ?ext preselecciona; sin parámetro, General
  • ext-ids.ts corrige el desajuste slug/id de 4 extensiones (se perdía atribución)
  • test con DOM mínimo: 5 casos, verde en EN y ES

Desplegado en producción por rsync (80 páginas, 0 borrados).

El parámetro ?ext existía pero no había forma de elegir extensión desde el formulario. - hidden -> select con 28 extensiones (nombre visible, id canónico) - ?ext preselecciona; sin parámetro, General - ext-ids.ts corrige el desajuste slug/id de 4 extensiones (se perdía atribución) - test con DOM mínimo: 5 casos, verde en EN y ES Desplegado en producción por rsync (80 páginas, 0 borrados).
feat(feedback): selector de extensión con preselección por ?ext
All checks were successful
CI / build (pull_request) Successful in 1m11s
Details
CI / build (push) Successful in 1m4s
Details
32bb6c13ae 
- el campo ext deja de ser un hidden y pasa a ser un <select> con las 28
  extensiones de la colección (nombre visible, id canónico como valor)
- al cargar con ?ext=... se preselecciona; sin parámetro queda en 'General'
- src/data/ext-ids.ts mapea slug del sitio -> id que envía setUninstallURL:
  colorpick, fontbuddy, printfriend y tabsnooze NO coinciden con su slug
  (color-pick, font-buddy, print-friend, tab-snooze) y se perdía la atribución
- tests/feedback-select.test.mjs ejecuta el script real del build con un DOM
  mínimo: 5 casos (id del navegador, slug del sitio, sin parámetro, slug nuevo)
All checks were successful
CI / build (push) Successful in 56s
Details
CI / build (pull_request) Successful in 54s
Details
This branch is already included in the target branch. There is nothing to merge.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/feedback-ext-selector:feat/feedback-ext-selector
git switch feat/feedback-ext-selector

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff feat/feedback-ext-selector
git switch feat/feedback-ext-selector
git rebase main
git switch main
git merge --ff-only feat/feedback-ext-selector
git switch feat/feedback-ext-selector
git rebase main
git switch main
git merge --no-ff feat/feedback-ext-selector
git switch main
git merge --squash feat/feedback-ext-selector
git switch main
git merge --ff-only feat/feedback-ext-selector
git switch main
git merge feat/feedback-ext-selector
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

   
feedback
No labels
bug
feedback
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
monyi/greasysprocket-website!3
No description provided.