Skip to content

Promoting upsells to callouts#24153

Merged
akristen merged 6 commits intodocker:mainfrom
akristen:cta-upsell
Feb 17, 2026
Merged

Promoting upsells to callouts#24153
akristen merged 6 commits intodocker:mainfrom
akristen:cta-upsell

Conversation

@akristen
Copy link
Contributor

Per feedback from leadership, promoting upsells to callouts and demoting other redundant banners.

@netlify
Copy link

netlify bot commented Feb 17, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 084b94d
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6994ee17d1306e00085f4771
😎 Deploy Preview https://deploy-preview-24153--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added the area/admin Relates to Docker Admin label Feb 17, 2026
@akristen akristen marked this pull request as ready for review February 17, 2026 21:38
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Documentation Review

Found 7 issues in the changed content:

  • 2 HIGH severity (information placement, content duplication)
  • 5 MEDIUM severity (typos, unclear instructions, redundancy)

All comments are on lines that were added or modified in this PR.


After accepting an invitation, you are now a member of the organization.

Invitation email links expire after 14 days. If your email link has expired, you can sign in to [Docker Hub](https://hub.docker.com/) with the email address the link was sent to and accept the invitation from the **Notifications** panel.
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Critical warning moved out of context

The information about invitation links expiring after 14 days was moved from DURING the step-by-step acceptance instructions (where it was a WARNING callout between steps 2-3) to AFTER the completion message.

Users need to know about the 14-day deadline WHILE they're following the invitation steps, not after they've finished. This time-sensitive information has reduced visibility in its current location.

Suggestion: Consider keeping this as a callout within or immediately before the acceptance steps, or at minimum, move it before the "After accepting an invitation" paragraph.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ignoring because if they're already following instructions to accept an invite, then they don't have to worry about the 14 day expiration.

Copy link
Contributor

@craig-osterhout craig-osterhout left a comment

Choose a reason for hiding this comment

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

LGTM. Left one non-blocking nit.

@akristen akristen merged commit 50f1c0b into docker:main Feb 17, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/admin Relates to Docker Admin area/billing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants