Skip to content

feat: migrate PackageIconUrl + publish docs site at toolkit.tharga.net#41

Merged
poxet merged 2 commits into
masterfrom
feature/package-icon-and-docs-site
Jun 8, 2026
Merged

feat: migrate PackageIconUrl + publish docs site at toolkit.tharga.net#41
poxet merged 2 commits into
masterfrom
feature/package-icon-and-docs-site

Conversation

@poxet

@poxet poxet commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Addresses two pending Internal requests from 2026-06-01:

  1. Move PackageIconUrl to thargelion.net/assets — replaces the WordPress-hosted URL in both csprojs with https://thargelion.net/assets/component-toolkit.png
  2. Documentation sites under tharga.net — adds a DocFX docs site published at toolkit.tharga.net

Also includes a routine NuGet package update across test projects.

What's in the PR

Icon

  • Tharga.Toolkit.csproj and Tharga.Toolkit.Standard.csproj → new PackageIconUrl
  • URL verified live (HTTP 200)

Docs site (docs/)

  • docfx.json — builds API reference from both csprojs
  • index.md + landing layout, 4 articles (getting-started, claims-and-identity, hashing, utilities)
  • Local docs/images/logo.png — option (a) to avoid the absolute-URL logo trap (matches MongoDB / Platform / Fortnox / Quilt4Net pattern)
  • templates/thg/public/main.css constrains navbar logo height
  • CNAMEtoolkit.tharga.net
  • /docs/_site, /docs/api, /docs/obj added to .gitignore

CI

  • Added docs + docs-deploy jobs to .github/workflows/build.yml, gated on master push + needs: release
  • Added pages: write and id-token: write permissions

README

Packages

  • Routine NuGet bumps in test projects (FluentAssertions 8.10, Microsoft.Extensions.* 10.0.8, Microsoft.NET.Test.Sdk 18.6, Tharga.Test.Toolkit 1.14.10, coverlet 10.0.1)

Test plan

  • Local build: 0 warnings
  • All 507 tests pass (322 Toolkit + 185 Standard)
  • Icon URL resolves: curl -I https://thargelion.net/assets/component-toolkit.png → 200
  • CI passes
  • After merge, the docs site builds and deploys (pending DNS for toolkit.tharga.nettharga.github.io)

Pre-reqs for go-live

  • DNS configuration for toolkit.tharga.net (one-time meta-repo work, out of scope for this PR)

Note

Same feature content as #40 (which targeted develop and has been merged there). This PR re-targets master directly per the corrected workflow — feature PRs go straight to master.

poxet added 2 commits June 8, 2026 09:53
PackageIconUrl:
- Both csprojs now point at https://thargelion.net/assets/component-toolkit.png
- Replaces the WordPress-hosted thargelion.se URL

Docs site:
- docs/ DocFX setup mirroring Tharga.Fortnox / Tharga.MongoDB pattern
  (option (a): local docs/images/logo.png to avoid the absolute-URL trap)
- CNAME -> toolkit.tharga.net
- Articles: getting-started, claims-and-identity, hashing, utilities
- API reference auto-generated from both Tharga.Toolkit and
  Tharga.Toolkit.Standard csprojs
- Navbar logo size constrained via templates/thg/public/main.css

CI:
- docs + docs-deploy jobs added to .github/workflows/build.yml,
  gated on master push and needs: release
- pages: write and id-token: write permissions added
- /docs/_site, /docs/api, /docs/obj gitignored

README updated with link to https://toolkit.tharga.net.
@codecov

codecov Bot commented Jun 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@poxet poxet merged commit 4769dd2 into master Jun 8, 2026
7 of 8 checks passed
@poxet poxet deleted the feature/package-icon-and-docs-site branch June 8, 2026 09:18
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