trent_larson/crowd-funder-for-time-pwa
4
1
Fork 2
Code Issues Pull Requests 22 Releases 5 Wiki Activity

fix(build): resolve web build script export error and add missing serve dependency #175

Open
anomalist wants to merge 0 commits from build-web-serve-test into master
pull from: build-web-serve-test
merge into: trent_larson:master
Conversation 1 Commits - Files Changed -
anomalist commented 2025-08-19 11:37:47 +00:00
Owner
Copy Link
  • Fix malformed multi-line comment in .env.test causing shell parsing failure
  • Add serve@14.2.4 as dev dependency to eliminate build-time installation prompts
  • Resolves "export: ' production).=': not a valid identifier" error
  • Test environment builds now complete successfully without user interaction

Fixes test environment build blocking issue

- Fix malformed multi-line comment in .env.test causing shell parsing failure - Add serve@14.2.4 as dev dependency to eliminate build-time installation prompts - Resolves "export: ' production).=': not a valid identifier" error - Test environment builds now complete successfully without user interaction Fixes test environment build blocking issue
anomalist added 1 commit 2025-08-19 11:37:48 +00:00
fix(build): resolve web build script export error and add missing serve dependency e3cc22245c 
- Fix malformed multi-line comment in .env.test causing shell parsing failure
- Add serve@14.2.4 as dev dependency to eliminate build-time installation prompts
- Resolves "export: ' production).=': not a valid identifier" error
- Test environment builds now complete successfully without user interaction

Fixes test environment build blocking issue
trentlarson commented 2025-08-23 01:39:27 +00:00
Owner
Copy Link

👍

👍
anomalist added 1 commit 2025-08-26 09:52:03 +00:00
Merge branch 'master' into build-web-serve-test 643f31c43a
anomalist added 1 commit 2025-08-26 09:57:43 +00:00
docs: enhance serve command documentation in BUILDING.md 427660d686 
- Clarify build:web:serve purpose as "production testing"
- Add "Why Use serve?" section explaining benefits
- Document SPA routing support for deep links (/discover, /account)
- Add dedicated "Local Serving with serve" technical section
- Explain server options (npx serve vs Python fallback)
- Improve developer understanding of when and why to use serve

Fixes documentation gap identified in serve command usage
anomalist added 12 commits 2025-08-27 09:54:23 +00:00
Fix: account for new Export Data dialog 9f976f011a 
- Stricter targeting of buttons since Register and Export Data dialogs appear on screen at the same time
- Locate success notification first since it appears first (and cannot be "clicked" through the overlapping dialog-overlay)
Merge branch 'master' into playwright-test-60-fix f31eb5f6c9
fix(DIDView): validate DID format before processing URL parameters 74c70c7fa0 
- Add DID validation using isDid() function to prevent invalid DIDs from loading current user's info
- Show error message and redirect to HomeView for invalid DID formats (e.g., /did/0)
- Import NOTIFY_CONTACT_INVALID_DID constant for consistent error messaging

Resolves: DIDView loading current user's info for invalid DID parameters
Merge branch 'master' into playwright-test-60-fix 0b9c243969
feat(harbor-pilot): add Playwright test investigation directive 4391cb2881 
- Create comprehensive MDC rule for systematic Playwright test failure investigation
- Integrate rule into harbor_pilot_universal.mdc for team-wide access
- Include investigation workflow, diagnostic commands, and evidence-based analysis
- Document specific failure patterns (alert stacking, selector conflicts, timing issues)
- Provide practical examples from recent test failure investigation
- Add investigation commands for error context, trace files, and page snapshots

This rule transforms ad-hoc test debugging into systematic investigation process,
leveraging Playwright's built-in debugging tools for faster root cause identification.
doc: add simpler instruction to get started with testing 623bf12ecd
fix: Fix onboard-meeting-members deep link with groupId. 9ea2f96106 
This was cherry-picked from master-patch
feat: for confirm list, allow to see 30 minutes before meeting start aa346a9abd
Merge branch 'master' into didview-invalid-did-handling baccb962cf
Test: change expected redirect view 1365adad92
Merge branch 'playwright-test-60-fix' into didview-invalid-did-handling 1a77dfb750
Merge pull request 'DIDView: validate DID format' (#176) from didview-invalid-did-handling into master 43e7bc1c12 
Reviewed-on: #176
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 build-web-serve-test:build-web-serve-test
git checkout build-web-serve-test
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
anomalist jose jsnbuchanan kentbull trentlarson
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: trent_larson/crowd-funder-for-time-pwa#175
Delete Branch "build-web-serve-test"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?