Skip to content

fix(plans): land trailing rubocop/self_serve commit from #15 onto main - #16

Merged
rrader26 merged 1 commit into
mainfrom
feat/plan-self-serve-portal-visibility
Jul 6, 2026
Merged

fix(plans): land trailing rubocop/self_serve commit from #15 onto main#16
rrader26 merged 1 commit into
mainfrom
feat/plan-self-serve-portal-visibility

Conversation

@rrader26

@rrader26 rrader26 commented Jul 6, 2026

Copy link
Copy Markdown

Why

PR #15 (feat/plan-self-serve-portal-visibility) was merged on 2026-07-01 18:14, but commit 71efb7e ("fix(plans): rubocop — order resolver args + simplify self_serve default") was pushed to the branch after the merge and never landed on main.

This trailing commit is what the prod lago-api image is actually running (20260701-thinkfleet-71efb7e), so main is currently behind prod. This PR reconciles them.

Changes (2 files, no migration)

  • available_plans_resolver.rb — reorder GraphQL args (rubocop, alphabetical)
  • plans/create_service.rb — simplify self_serve default (nil? ? true : xx != false; identical truth table)

Discovered while diagnosing a prod login-flapping incident where prod was found to be running off-main builds.

@rrader26
rrader26 merged commit 96142d2 into main Jul 6, 2026
2 of 13 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.

2 participants