Skip to content

Polish the 26Q3.5 release landing page - #47

Merged
MstfSlm38 merged 1 commit into
26Q2.5from
fix/maintenance-countdown-date
Nov 4, 2026
Merged

MstfSlm38 merged 1 commit into
26Q2.5from
fix/maintenance-countdown-date

Conversation

@MstfSlm38

Copy link
Copy Markdown
Collaborator

Summary

  • Add a fixed Liquid Glass navigation bar
  • Add Turkish, English, and Spanish language switching
  • Add light and dark theme switching with saved preferences
  • Expand the release page with additional sections and apply the CodeSnip accent palette throughout

Notes

  • Source branch: fix/maintenance-countdown-date
  • Target branch: 26Q2.5
  • Direct commits to the target branch are blocked by repository rules, so these changes are submitted through a pull request.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-04T14:18:09.688345Z 1235904 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@MstfSlm38
MstfSlm38 merged commit b96a149 into 26Q2.5 Nov 4, 2026
1 check passed
@chatgpt-codex-connector

Copy link
Copy Markdown

💡 Codex Review

"version": "26Q3.5",

P1 Badge Normalize the version before building release artifacts

The committed version 26Q3.5 is not SemVer, and the tag-triggered release.yml runs npm run dist:win without the version-normalization step used in ci.yml. Windows packaging therefore fails while parsing the app version, before producing an .exe, so the release job cannot create releases. Electron Builder documents this Windows SemVer requirement.


JSON.parse(rawData);

P2 Badge Validate the imported backup's schema

If a user selects syntactically valid JSON that is not an array of snippet records, this check still succeeds; the renderer then overwrites all_snippets_data with that content and reloads. On startup, snippet rendering expects an array and throws, leaving the existing collection overwritten. Validate the backup's expected structure before returning success.



P2 Badge Do not gate publishing on an undefined test script

On each release: created event, this step runs before publish-npm, but package.json defines no scripts.test; npm test exits with a missing-script error and fails the build job, skipping publication. Add a test script or remove the test step. npm's documentation says npm test runs the command specified by scripts.test.

ℹ️ 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".

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