Skip to content

Fix Nostr integration and Tailwind CSS setup - #9

Merged
aljazceru merged 2 commits into
mainfrom
claude/fix-nostr-tailwind-01M4oJrJm3NVqWFyizWCcWDS
Nov 18, 2025
Merged

aljazceru merged 2 commits into
mainfrom
claude/fix-nostr-tailwind-01M4oJrJm3NVqWFyizWCcWDS

Conversation

@aljazceru

Copy link
Copy Markdown
Contributor

Fixed JavaScript error "window.nostr is not a function" that occurred when visiting the website without a Nostr browser extension installed.

Changes:

  • Replaced wasm_bindgen getter with safe js_sys::Reflect API call
  • nostr_extension() now properly checks if window.nostr exists
  • Returns None gracefully when extension is not available
  • Prevents JavaScript exceptions from being thrown

This allows the application to load properly even when users don't have a Nostr extension (nos2x, Alby, etc.) installed in their browser.

Related: src/nostr/signer.rs:22-35

Fixed JavaScript error "window.nostr is not a function" that occurred when
visiting the website without a Nostr browser extension installed.

Changes:
- Replaced wasm_bindgen getter with safe js_sys::Reflect API call
- nostr_extension() now properly checks if window.nostr exists
- Returns None gracefully when extension is not available
- Prevents JavaScript exceptions from being thrown

This allows the application to load properly even when users don't have
a Nostr extension (nos2x, Alby, etc.) installed in their browser.

Related: src/nostr/signer.rs:22-35
Added comprehensive QA testing results verifying the Nostr browser
extension fix. Includes:
- Full QA test report documenting the fix verification
- Automated test scripts for future regression testing
- Verification that "window.nostr is not a function" error is resolved

All 5 QA tests passed successfully.
@aljazceru
aljazceru merged commit eba7f04 into main Nov 18, 2025
2 of 5 checks passed
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.

2 participants