Skip to content

Fix: rebrand the license page to Nexcess Licensing - #206

Merged
dpanta94 merged 10 commits into
mainfrom
fix/nexcess-licensing-name
Oct 12, 2026
Merged

dpanta94 merged 10 commits into
mainfrom
fix/nexcess-licensing-name

Conversation

@jonwaldstein

@jonwaldstein jonwaldstein commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Description

Harbor's license page is now called "Nexcess Licensing", and the "Liquid Web" copy around it now says Nexcess. This replaces #204, which used "Unified License Manager".

  • The page title, the welcome screen heading, the legacy license notice and every error message on the page say "Nexcess Licensing".
  • The Settings menu item is now Nexcess Licensing (was "Liquid Web Products"), and the license link reads "Manage license in Nexcess".
  • The page header shows the Nexcess logo instead of the Liquid Web one.
  • The legacy license notice says the plugin is "now part of Nexcess's software offerings".
  • The welcome screen drops the "Liquid Web by Nexcess" line, since the heading already says Nexcess.
  • The integration guide, skill/SKILL.md and the AGENTS.md snippet in the README now tell plugins to say "Nexcess Licensing". The old "brand-neutral" reasoning is gone.
  • The page URL and the "Licensing" item in plugin menus stay as they are.

Testing Instructions

  1. Look under Settings. The menu item reads "Nexcess Licensing".
  2. Open it with no license saved. The welcome screen heading reads "Nexcess Licensing" with no "Liquid Web by Nexcess" line under it. The browser tab title also reads "Nexcess Licensing".
  3. Save a license. The page header shows the Nexcess logo, and the link under the license card reads "Manage license in Nexcess".
  4. Open a plugin's legacy license page that shows the notice. It says "now part of Nexcess's software offerings", and the link at the end reads "Nexcess Licensing".
  5. Block the REST requests in dev tools and reload the license page. The error message starts with "Nexcess Licensing failed to…".
  6. bun run test:js and slic run wpunit cover the renamed strings.

Pre-review Checklist

  • Acceptance criteria satisfied and tested
  • Changelog entry added
  • E2E or unit tests updated

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the Harbor integration reference to use “Nexcess Licensing page,” matching the terminology used elsewhere in the document.
  • Tests
    • Updated the end-to-end test suite’s section label to “Nexcess Licensing page.” Test setup and coverage are unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Enterprise
  • Run ID: 9b3daabe-fa1c-4caa-9ee2-b80bf0f7ebd8

📥 Commits

Reviewing files that changed from the base of the PR and between d38a9ef and 48d6e0f.


📒 Files selected for processing (2)
  • skill/SKILL.md
  • tests/e2e/software-manager.spec.ts

🚧 Files skipped from review as they are similar to previous changes (1)
  • tests/e2e/software-manager.spec.ts

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.



📝 Walkthrough

Walkthrough

The skill description and Playwright suite label now use “Nexcess Licensing page.” The suite setup and tests are unchanged.

Changes

Nexcess Licensing naming references

Layer / File(s) Summary
Update licensing page references
skill/SKILL.md, tests/e2e/software-manager.spec.ts
The skill description and E2E suite label now refer to the “Nexcess Licensing page.” Test setup and cases remain unchanged.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Suggested reviewers: dpanta94


Merge Risk: ⚪ Minimal · up to 48d6e

The reviewed changes update naming references without changing behavior, so no material merge risk is identified.

Pre-merge checks | Passed 4 | Failed 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage Warning Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 16 functions across 25 files. (1 skipped:… Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: rebranding the license page to “Nexcess Licensing.”
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 16 functions across 25 files. (1 skipped: 1 unsupported.)


  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR

🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

jonwaldstein and others added 2 commits October 8, 2026 20:58
@jonwaldstein jonwaldstein changed the title Fix: call the license page Nexcess Licensing Fix: rebrand the license page to Nexcess Licensing Oct 8, 2026
jonwaldstein and others added 3 commits October 8, 2026 21:01
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@jonwaldstein jonwaldstein added the playground Build a WordPress Playground preview for this pull request label Oct 8, 2026
jonwaldstein and others added 2 commits October 8, 2026 21:06
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

WordPress Playground Preview

Try this pull request in a throwaway WordPress in your browser. Harbor Dev Tools runs this branch of Harbor as the leader, licensed with the lwsw-unified-pro-2026 fixture key. Free features can be enabled; premium ones cannot, because their downloads need a real key. Built 2026-10-12 15:15 UTC from 48d6e0f, or download the zip for a site of your own.

Harbor Dev Tools only

Open WordPress Playground Preview

Each button below also installs the latest release of one plugin beside Harbor Dev Tools. That plugin bundles its own copy of Harbor, and Harbor Dev Tools stays the leader.

With GiveWP

Open WordPress Playground Preview with GiveWP

With The Events Calendar

Open WordPress Playground Preview with The Events Calendar

With Kadence Blocks

Open WordPress Playground Preview with Kadence Blocks

Main's software-stack entry already announces the Nexcess Licensing rename, so a second entry would repeat it in the release notes.

@dpanta94 dpanta94 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you @jonwaldstein - most of the changes here have been completed as part of #205

@dpanta94 dpanta94 self-assigned this Oct 12, 2026
@dpanta94
dpanta94 merged commit 97c18c9 into main Oct 12, 2026
26 checks passed
@dpanta94
dpanta94 deleted the fix/nexcess-licensing-name branch October 12, 2026 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

playground Build a WordPress Playground preview for this pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants