-
Notifications
You must be signed in to change notification settings - Fork 0
Add a reusable activation URL API #181
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
dave-green-uk
wants to merge
33
commits into
bucket/activation-flow-api
from
smtnc-1844-harbor-expose-a-reusable-activation-url-api-for-php-and-js
Open
Changes from 22 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
5af561a
Add a reusable activation URL API for PHP and JS
dave-green-uk af49f7d
Build frontend assets
dave-green-uk d97364c
Fix the default activation redirect landing on a WordPress error page
dave-green-uk 7de8400
Correct the enqueue timing guidance for the activation script
dave-green-uk d3fc2b1
Correct the rationale for the canonical redirect URL
dave-green-uk bd1a42a
Use US spelling in the activation URL test docblock
dave-green-uk f4b36ce
Refresh licensing data when the portal returns a user to the site
dave-green-uk d78dad3
Fix fallout from moving the refresh out of the admin page
dave-green-uk 1c8efec
Realign the constructor docblock after dropping a parameter
dave-green-uk 0a4a279
Check the return tag before resolving the refresh handler
dave-green-uk dc9f24f
Expose activation URLs through stable global functions
dave-green-uk 006b85b
Align the activation-URL doc tables to satisfy markdownlint
dave-green-uk 081c461
Address review feedback on the activation URL API
dave-green-uk 189b9c1
Stop suppressing exit() in the test suite
dave-green-uk cf6ed65
Fix the redirect stand-in's class scope and a docblock tag
dave-green-uk a5f52f1
Warn at runtime when the deprecated redirect handler is called
dave-green-uk 7c0651b
Merge bucket/activation-flow-api into the activation URL API branch
dave-green-uk bc40333
Address the second review round on the activation URL API
dave-green-uk e203f94
Build frontend assets
dave-green-uk 38455b8
Isolate the enqueue hook in the script dependency tests
dave-green-uk f981248
Return null rather than an empty string when there is no activation URL
dave-green-uk 6ebaeef
Expose product license lookups so consumers need not touch internal c…
dave-green-uk e58fb2f
Drop the browser activation helper in favour of building URLs in PHP
dave-green-uk 11d89ed
Let the return handler decide whether a request is a return trip
dave-green-uk ae32efd
Cover the compiled asset resolver with unit tests
dave-green-uk 4067013
Drop the changelog note for an argument that never shipped
dave-green-uk 2c50995
Qualify the base activation URL function as product activation
dave-green-uk f8df048
Fix two spellings cspell rejects
dave-green-uk 4735e06
Inject the catalog repository rather than service-locating it
dave-green-uk a52253b
remove LiquidWeb\Harbor\Utils\Assets class
pramodjodhani d6ebc7b
Merge branch 'smtnc-1844-harbor-expose-a-reusable-activation-url-api-…
pramodjodhani 6375aac
remove AssetsTest.php
pramodjodhani 96ade84
repalce maybe_refresh_after_activation with closure/anon funciton
pramodjodhani File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| <?php return array('dependencies' => array(), 'version' => '0658b3a3cd288e7d5743'); |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| <?php return array('dependencies' => array(), 'version' => '9221f4ca9e52af97fee3'); |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| significance: minor | ||
| type: feature | ||
| entry: Licensing data is now refreshed automatically when the Liquid Web portal returns a user to the site after activating, so screens gated on license state are correct on arrival. Host plugins need no code for this; sending the user through an activation URL is the whole opt-in. This supersedes the Software Manager page's ?refresh=auto handler, which only covered that one screen; Feature_Manager_Page::maybe_redirect_after_refresh() is deprecated and no longer hooked | ||
| timestamp: 2026-07-22T00:00:00.000Z |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| significance: minor | ||
| type: feature | ||
| entry: Added a reusable activation URL API. Host plugins can build Liquid Web portal activation URLs from PHP via the lw_harbor_get_activation_base_url() and lw_harbor_get_product_activation_url() global functions, or in the browser via the shared activation script, which exposes window.lwHarbor.buildActivationUrl() and is attached to a consumer's own script with lw_harbor_add_activation_script_dependency() | ||
| timestamp: 2026-07-22T00:00:00.000Z |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| significance: minor | ||
| type: feature | ||
| entry: Added lw_harbor_is_product_licensed() and lw_harbor_get_product_tier() so host plugins can ask whether a license covers a product, and at which tier, without resolving Harbor's internal licensing classes from their own vendor-prefixed copy. The tier argument to lw_harbor_get_product_activation_url() is now optional and accepts null, which sends an unscoped sku so the portal offers its own product and tier picker; get_product_tier() returns null when a license covers a product at several tiers, letting that choice fall to the portal rather than being guessed | ||
| timestamp: 2026-07-30T00:00:00.000Z |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.