Skip to content

fix: unify version tags to physics-guard and restore release 1.6 badge - #137

Merged
Coding-Dev-Tools merged 7 commits into
mainfrom
fix/version-tags-physics-guard
Aug 10, 2026
Merged

fix: unify version tags to physics-guard and restore release 1.6 badge#137
Coding-Dev-Tools merged 7 commits into
mainfrom
fix/version-tags-physics-guard

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

OneDrive sync keeps reverting version tags from physics-guard to pin-only-physics. This PR restores the correct tags and also fixes the release_version and plan badge logic that was lost in a prior revert.

- Version tags: pin-only-physics → physics-guard across all dashboard assets
- Restore release_version='1.6' in savings preset query
- Restore plan badge logic in ledger.js

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 51ab802e30

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread engraphis/dashboard_assets/ledger.js Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: af412514d8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread engraphis/dashboard_assets/ledger.js Outdated
Coding-Dev-Tools and others added 3 commits August 9, 2026 22:53
Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
updatePlanBadge() and its click routing were restored with the 1.6
release, but dashboard_assets/index.html no longer contains the
#plan-badge element, so the badge logic never rendered for any license.
Re-add the badge anchor in the sidebar brand row (the CSS rules already
exist) so plan, trial, and billing state is visible again.

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: aa75c3adf5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread engraphis/dashboard_assets/index.html
…ing tests

The plan-badge element is restored in the Ledger sidebar, so the e2e
assertions flip from toHaveCount(0) to expecting the badge with its
active-license label (PRO / TRIAL).

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
@Coding-Dev-Tools
Coding-Dev-Tools merged commit 128fe05 into main Aug 10, 2026
23 checks passed
@Coding-Dev-Tools
Coding-Dev-Tools deleted the fix/version-tags-physics-guard branch August 10, 2026 13:08
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