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()`.
The role index page now uses a Vue component instead of server-rendered Blade links. Update test to check for the HTML-encoded JSON data in the component props instead of rendered anchor tags. 🤖 Generated with [Claude Code](https://claude.com/claude-code) 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>
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Reverts accidental port change that broke CI. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This reverts commit d60519e.
- 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>
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>
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>
- 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>
|
79 tasks
Bring in 30 commits from develop (PRs #817-#822 and related): - Wiki login fallback when wiki unavailable (#819) - Group edit map in wrong place fix (#818) - Configurable support email address (#821) - HTML escaping fix in user list view - Intermittent Playwright test fixes (device, group invitation, notification) - SQL precedence fix in UserGroups::scopeConfirmedInvitation - Bitnami → official/Bitnami Legacy image swaps for Discourse stack - French-Belgian postcode translation tweak (#822) Conflict resolutions: - .circleci/config.yml: kept L10's machine executor + Task/Docker Compose paradigm; develop's multi-image executor changes are superseded. - docs/local-development.md: kept L10's collapsible-sections structure; WSL content is present on both sides. - phpunit.xml: kept L10's DB_TEST_DATABASE=restarters_db_test (separate test DB is the intended L10 setup). - tests/Integration/utils.js: took develop's more robust dropzone wait (expect .dz-success visible, 30s timeout) over L10's preview+sleep. - resources/js/translations.js: deleted (Vite laravel-translator plugin now generates it); develop's edit discarded. - package-lock.json: regenerated via npm install against merged package.json.
The merge took develop's .dz-success selector for the photo upload wait, but the L10 branch dropzone DOM doesn't produce .dz-success. Revert to L10's .dz-image img selector. Keep develop's 30s timeout (was 10s on L10) since it's more forgiving without cost.
|
| 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'@'%';" |
| - 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;" |
| - 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',''));" |
| echo "Waiting for services in profile: {{.PROFILE}}" | ||
| echo "" | ||
|
|
||
| # Wait for core services (always needed) |
edwh
added a commit
that referenced
this pull request
Apr 23, 2026
Bring in the Laravel 10 upgrade (via PR #816 merge) plus develop drift. Single conflict in app/Listeners/LogInToWiki.php constructor signature: kept develop's ?UserCreator $mediawikiUserCreator (nullable type, no default) over the older UserCreator $mediawikiUserCreator = null form. Both work with the MediawikiServiceProvider binding that returns null when Wiki is unavailable; keeping develop's form for consistency with the tested wiki-fallback flow. package-lock.json regenerated via npm install.
Closed
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
4 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.
Overview
This PR builds on #806 (the initial Laravel 10 upgrade via Laravel Shift) and adds substantial additional work to complete the modernization.
Changes Since PR #806
Build System Overhaul
@vite()directive instead ofasset()PHP & Framework
jQuery Plugin Removal & Vue Replacements
Removed heavy jQuery dependencies and replaced with lighter alternatives:
Testing & CI
Development Environment
Bug Fixes
Statistics
Requirements After Merge
composer installandnpm installnpm run buildornpm run dev