Skip to content

fix: add Firebase Storage read rule for tilesets#219

Merged
enspyr-admin merged 1 commit intomainfrom
fix/tileset-storage-rules
Apr 9, 2026
Merged

fix: add Firebase Storage read rule for tilesets#219
enspyr-admin merged 1 commit intomainfrom
fix/tileset-storage-rules

Conversation

@maxwell-merge-slam
Copy link
Copy Markdown
Contributor

Summary

  • Root cause of LimeZu wall tileset download failure: no storage rule for tilesets/ path
  • Added authenticated read rule for tilesets/{tilesetFile}
  • Added storage config to firebase.json so rules are deployable
  • Rules already deployed to production

Test plan

  • firebase deploy --only storage succeeded
  • Docs-only change (storage rules + firebase config), no Dart code changed

🤖 Generated with Claude Code

Tileset downloads were denied because storage.rules had no rule for
the tilesets/ path. Add authenticated read access for tileset images.
Also add storage config to firebase.json so rules can be deployed.

Co-Authored-By: Claude <noreply@anthropic.com>
Copy link
Copy Markdown

@kelvin-bit-brawler kelvin-bit-brawler bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The missing piece. Without this rule, every tileset download silently fails.

@enspyr-admin enspyr-admin enabled auto-merge (squash) April 6, 2026 14:04
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

LCOV of commit 842a330 during Test and Preview on PR #280

Summary coverage rate:
  lines......: 63.9% (3372 of 5280 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

Visit the preview URL for this PR (updated for commit 842a330):

https://adventures-in-tech-world-0--pr219-fix-tileset-storage-uprztr85.web.app

(expires Mon, 13 Apr 2026 14:12:31 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 2428000e66ed21b952892aaec940ffabd9f9a921

@enspyr-admin enspyr-admin merged commit 1582b5b into main Apr 9, 2026
4 checks passed
@enspyr-admin enspyr-admin deleted the fix/tileset-storage-rules branch April 9, 2026 10:34
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