Skip to content

feat: Support on Ko-fi link in the site footer and extension options - #102

Open
ajithakdev wants to merge 1 commit into
mainfrom
feat/ko-fi-support
Open

ajithakdev wants to merge 1 commit into
mainfrom
feat/ko-fi-support

Conversation

@ajithakdev

Copy link
Copy Markdown
Owner

Plain links, no Ko-fi script, so the site CSP and the extension's permissions are unchanged. The extension change needs a store upload with the Sunday release. Also updates docs/SITE-PLAN.md for the portfolio repo rename (s-portfolio is now ajithakdev.com). CHANGELOG and RESOURCES-AND-GUIDE updated.

Lint: 0 errors (6 warnings, unchanged). Tests and docs checks pass.

@ajithakdev

Copy link
Copy Markdown
Owner Author

Pre-merge checklist

Addressed

  • "Support on Ko-fi" link in the site footer (every page through _partials/footer.html)
  • One line with a Ko-fi link on the extension's options page
  • No Ko-fi script; the site CSP and the extension's permissions are unchanged

Tested

  • npm run build passes; the link is in dist/index.html, about/ and 404.html
  • npm test passes, docs checks PASS (19)
  • npm run lint: 0 errors; the 6 warnings come from JS files this PR doesn't touch
  • Options page opened in a loaded extension

CI and release

  • check passes
  • Sunday release: tag, wrangler deploy, store uploads (the extension change needs the upload)

Docs

  • CHANGELOG line under Unreleased
  • RESOURCES-AND-GUIDE.md Ko-fi row; SITE-PLAN.md renamed for ajithakdev.com

After merge: goes out with Sunday's release.

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.

1 participant