Skip to content

Remove AllowMeterOverage and CanShowBillingIssueBanner#40

Merged
BrewingCoder merged 3 commits intomainfrom
issue-33-remove-allow-meter-overage
Mar 20, 2026
Merged

Remove AllowMeterOverage and CanShowBillingIssueBanner#40
BrewingCoder merged 3 commits intomainfrom
issue-33-remove-allow-meter-overage

Conversation

@BrewingCoder
Copy link
Owner

Summary

  • Removes Workspace.AllowMeterOverage field and AllWorkspaceSettings.CanShowBillingIssueBanner field from model.go
  • Removes updateAllowMeterOverage mutation from GraphQL schema and resolver
  • Removes allow_meter_overage from Workspace GraphQL type
  • Removes field mapping in workspace copy helper (resolver.go)
  • Removes CanShowBillingIssueBanner: false from EnableAllWorkspaceSettings
  • Regenerated GraphQL code via gqlgen

Net -273 lines of dead billing/metering code.

Closes #33

Test plan

  • go build ./... passes
  • go test ./model/ — 206 tests pass
  • gofmt clean
  • CI: backend build + lint + test
  • CI: SonarQube + CodeQL

🤖 Agentic: submitted autonomously by Claude Code agent

🤖 Generated with Claude Code

BrewingCoder and others added 3 commits March 19, 2026 18:57
Dead SaaS billing fields — HoldFast has no metering or billing banners.
Removes Workspace.AllowMeterOverage, AllWorkspaceSettings.CanShowBillingIssueBanner,
UpdateAllowMeterOverage mutation from GraphQL schema and resolver,
and field mapping in workspace copy helper.

Regenerated GraphQL code via gqlgen.

Closes #33

🤖 Agentic: submitted by Claude Code agent

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@BrewingCoder BrewingCoder merged commit c18daed into main Mar 20, 2026
10 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.

Remove AllowMeterOverage and CanShowBillingIssueBanner

1 participant