fix: allow full PDF download via free query param#377
Conversation
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
Restore the ability to generate a full PDF when the ?free query parameter is present in the URL. Uses the `in` operator to check for parameter presence rather than its value, so ?free, ?free=, and ?free=1 all work correctly.
3f04efb to
be17c6d
Compare
Firebase CLI v15+ requires Node.js >= 20.
|
Visit the preview URL for this PR (updated for commit a69eb0b): https://whatsanalyze-80665--pr377-fix-free-query-param-34vsso6f.web.app (expires Sun, 22 Mar 2026 17:52:25 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 5796a863d3a1b99df3fcb31feec7a39514d97aa4 |
Replace ?. and ?? with plain JS to ensure compatibility with older mobile Safari versions that Nuxt 2 may not transpile for.
Summary
?freequery parameter is present in the URLinoperator to check for parameter presence rather than truthiness, so?free,?free=, and?free=1all work correctly$routeorquerywith optional chaining and nullish coalescingTest plan
?free→ sample button should generate a sample PDF?free→ sample button should generate a full PDF?free=→ full PDF?free=1→ full PDF