Skip to content

Conversation

@escapedcat
Copy link
Contributor

Fixes: #576

Add new page at /tagger-onboarding for prospective taggers to submit their name and email. Creates a Gitea issue with type/tagger-onboarding label. Added navigation link in Maintain dropdown.

🤖 Generated with Claude Code

Add new page at /tagger-onboarding for prospective taggers to submit
their name and email. Creates a Gitea issue with type/tagger-onboarding
label. Added navigation link in Maintain dropdown.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@netlify
Copy link

netlify bot commented Dec 16, 2025

Deploy Preview for btcmap ready!

Name Link
🔨 Latest commit 1330423
🔍 Latest deploy log https://app.netlify.com/projects/btcmap/deploys/6941829d30c1710008952056
😎 Deploy Preview https://deploy-preview-584--btcmap.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 72 (🔴 down 23 from production)
Accessibility: 97 (no change from production)
Best Practices: 92 (🔴 down 8 from production)
SEO: 100 (no change from production)
PWA: 90 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Update add-location and verify-location labels to use type/ prefix:
- location-submission → type/add-location
- location-verification → type/verify-location

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@dadofsambonzuki
Copy link
Member

I think it's better to switch label assignment away from names to IDs.

location-submission is 901
location-verification is 903

That way we can edit the labels in GT without breaking stuff.

tagger-onboarding is 1410 over on the private infra repo (as we want to protect email addresses etc) - https://gitea.btcmap.org/teambtcmap/btcmap-infra/. We'll probably need to change the gitea helper to deal with multiple repos.

escapedcat and others added 6 commits December 16, 2025 14:24
Change "View all" link from GitHub to Gitea URL and update text.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Switch from label names to label IDs for issue creation
- Add GITEA_LABELS constant object grouped by repository
- Support btcmap-infra repo for tagger-onboarding (private)
- Remove unused getLabels() and createLabel() functions
- Remove area label logic from all endpoints
- Revert ticket filters to original label names

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add getLabels and createLabel functions with multi-repo support
- Update createIssueWithLabels to accept optional areaLabels parameter
- Restore area lookup logic in add-location and verify-location endpoints
- Include associated areas in issue body text

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>


- Create unified endpoint handling all form submissions
- Config-driven repo/label selection per form type
- Shared captcha validation logic
- Delete 4 individual endpoint files
- Update all form pages to use new endpoint

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Use satisfies + keyof typeof to derive type from data

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@escapedcat escapedcat requested a review from Copilot December 16, 2025 16:05

This comment was marked as resolved.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Tagger Onboarding Page/Form

3 participants