Skip to content

fix(app): use bundled Runbox 7 favicon#1843

Open
Techniacus wants to merge 1 commit into
runbox:masterfrom
Techniacus:patch-1
Open

fix(app): use bundled Runbox 7 favicon#1843
Techniacus wants to merge 1 commit into
runbox:masterfrom
Techniacus:patch-1

Conversation

@Techniacus
Copy link
Copy Markdown

Updates the Runbox 7 web app favicon to use the bundled current Runbox 7 icon instead of the older external /_img/index/logobox.png asset.

Closes #922.

Verification

  • npm.cmd run lint passed with existing warnings only
  • npm.cmd run policy passed with existing commit-history warnings only
  • npx.cmd ng build --configuration production --base-href=/app/ runbox7 passed
  • Confirmed generated dist/index.html references assets/icons/icon-192x192.png
  • Confirmed dist/assets/icons/icon-192x192.png exists

Bounty

This addresses Runbox 7 good-first-issue #922, which appears eligible for the Runbox 7 Feature and Bug Bounty Program's minor-feature reward tier.

AI assistance disclosure

I used Codex to inspect the issue, prepare the patch, and summarize test results. I reviewed the generated changes before submission.

Use the bundled Runbox 7 icon asset for the web app favicon.

AI assistance disclosure: I used Codex to inspect the issue, prepare the patch, and summarize test results. I reviewed the generated changes before submission.
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.

web app: favicon

1 participant