RES-2054: Group Tags - Network Coordinator Management#829
Open
RES-2054: Group Tags - Network Coordinator Management#829
Conversation
…laywright packages.
…ssions we don't have in Docker Compose environment.
Since PHP 5.4 the short array syntax `[]` may be used instead of `array()`.
- Taskfile.yml: Update wait_for_service check to use port 8026 - CLAUDE.md: Update Mailhog URL - docs/local-development.md: Update Mailhog URL 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Refactored to recursively process nested translation structures instead of failing when encountering arrays. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Logs EventsUsers status and role to help diagnose why attending is sometimes true when it should be false in CI. Also adds test artifacts to .gitignore. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add fr and fr-BE translations for new network strings - Remove unused translation keys (general.count, show.about_modal_header) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Use "Etiquette" (without accent) to match existing usage - Use "Repair Café" instead of "groupe" for consistency 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Removed fwrite(STDERR) which was causing TeamCity to mark the test as failed even when assertions passed. Debug info now only appears in assertion failure messages. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- /api/v2/networks/{id}/tags returns empty for unauthenticated users
- /api/v2/groups/tags returns empty for unauthenticated users
- Added tests for both endpoints
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The test occasionally fails in CI with all_confirmed_restarters_count being 1 instead of 0. Added debug info to show group membership state when assertion fails. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
After making unauthenticated API calls return empty tags, the test needs to authenticate first to see the tags. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The test expected unauthenticated users to see network tags, but we changed the API to return empty for unauthenticated users. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Must be authenticated to see tags after unauthenticated API change. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Center title and intro text on landing page - Reduce line-height on h1 for tighter title spacing - Add white-space: nowrap to stat headers to prevent line breaks - Add missing description field to skills seeder (required NOT NULL column) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Use more specific selector (label.btn.btn-checkbox) for higher specificity - Override background color to gray (#E4E4E4) - Remove uppercase text-transform - Add background-image: none to prevent gradient overrides Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add docker:vite, docker:vite:start, docker:vite:stop tasks - Change HMR host from hardcoded www.example.com to env variable with localhost default - Update CLAUDE.md with critical Docker task command documentation Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Create EmailValidation.vue component that checks if email already exists on blur and displays validation message. Replaces inline HTML with proper Vue component following codebase patterns. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The templates use kebab-case (<categories-table>, <roles-table>) but components were registered without hyphens. Vue's template-to-component resolution requires matching names. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Past events were missing timezone in the expanded event data, causing date/time display issues on group pages. Now explicitly passing timezone alongside event_date_local, start_local, and end_local. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The route was incorrectly placed in the ensureAPIToken middleware group but requires authentication since it calls auth()->user(). Moved to the auth+verifyUserConsent+ensureAPIToken group where it belongs. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Hide hosts section in EventDetails when no hosts assigned - Update StatsShare to use __() instead of $lang.get/choice methods Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
The checkbox in the event invite modal was not functional - clicking it did nothing. This fix: 1. Adds server-side code to pass group member emails as a data attribute on the checkbox element 2. Adds JavaScript handler to populate the email textarea when the checkbox is checked, and remove them when unchecked 3. Properly enables the submit button when emails are present Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Create EventInviteModal.vue component to replace blade modal - Remove jQuery handler from app.js and use Vue event handling - Add conditional email field to Volunteer API resource (only visible to hosts, network coordinators, and admins for privacy) - Update EventAttendance.vue to use new Vue modal component - Remove blade modal include from events/view.blade.php - Add translation for manual_invite_placeholder in en/fr/fr-BE - Add tests for volunteer email visibility based on user role Fixes the "Add invites for group members" checkbox not populating emails when clicked. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Event invite modal: Add Tab key support for email tags, fix CSRF token retrieval from meta tag, update placeholder translations - Notifications: Fix mark as read using event delegation for dynamic elements - Profile skills: Add 'Ctrl-click to select multiple' hint, hide dropdown arrow - Landing page: Fix 'Waste prevented' text wrapping with proper styling Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Brings in post-release fixes: - Event invite modal converted to Vue component with Tab key support - Fix notifications mark as read (event delegation) - Fix skills list dropdown arrow and add Ctrl-click hint - Fix landing page 'Waste prevented' layout wrapping Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add edit functionality for network tags (PUT endpoint, modal, tests) - Filter global tags from NC view on group edit page - Preserve global/other network tags when NC saves group - Add GroupTagsBadges component to display tags on group view - Rename API parameter 'tag' to 'group_tag' for clarity - Fix memory exhaustion in loginRegisterStats (use COUNT query) - Fix stats calculation in Network model (aggregate SQL queries) - Add translations for tag editing UI - Update tests and regenerate OpenAPI spec Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add group_tags to Group model's $with array for eager loading - Update GroupTagsBadges to handle both 'tags' and 'group_tags' keys - Handle both 'name' and 'tag_name' property names for display Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
NCs should not see global tags from the API - this is a privacy issue. The getFilteredTagsForUser method now filters the tags returned by the Group resource based on the authenticated user's role. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Override toArray() to filter group_tags based on user permissions when the model is serialized to JSON in Blade templates. This ensures NCs don't see global tags on the group view page. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
3 tasks
|
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| name: Setup application | ||
| command: | | ||
| # Grant timezone access - run directly on MySQL container | ||
| docker exec restarters_db mysql -u root -ps3cr3t -e "GRANT SELECT ON mysql.time_zone_name TO 'restarters'@'%';" |
Check failure
Code scanning / SonarCloud
MySQL database passwords should not be disclosed
| - run: sed -i 's/HONEYPOT_DISABLE=.*$/HONEYPOT_DISABLE=TRUE/g' .env | ||
| # Setup additional configuration needed for CI (most setup already done by docker_run.sh) | ||
| # Set MySQL function creators using session variable (compatible with MySQL 5.7) | ||
| docker exec restarters_db mysql -u root -ps3cr3t -e "SET GLOBAL log_bin_trust_function_creators = 1;" |
Check failure
Code scanning / SonarCloud
MySQL database passwords should not be disclosed
| - run: npx playwright install-deps | ||
| - run: npm install -D @playwright/test | ||
| # Disable ONLY_FULL_GROUP_BY for compatibility with getItemTypes() query | ||
| docker exec restarters_db mysql -u root -ps3cr3t -e "SET GLOBAL sql_mode=(SELECT REPLACE(@@sql_mode,'ONLY_FULL_GROUP_BY',''));" |
Check failure
Code scanning / SonarCloud
MySQL database passwords should not be disclosed
| echo "Waiting for services in profile: {{.PROFILE}}" | ||
| echo "" | ||
|
|
||
| # Wait for core services (always needed) |
Check failure
Code scanning / SonarCloud
MySQL database passwords should not be disclosed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/api/v2/networks/{id}/tagsAlso includes merge of Laravel 10 upgrade branch.
Related
Test plan
🤖 Generated with Claude Code