Skip to content

Conversation

@JakeSCahill
Copy link
Contributor

@JakeSCahill JakeSCahill commented Nov 26, 2024

Description

Review deadline: 2 December

Addresses recent feedback on the licensing doc:

We should clarify. This table is what happens automatically upon expiry, but there are also more restrictions after expiry where you can't enable any enterprise feature.

Also we need to add that Console (on expiry) forwards you to a landing page if CONSOLE RBAC or CONSOLE SSO (not cluster RBAC) is enabled.

Page previews

https://deploy-preview-878--redpanda-docs-preview.netlify.app/24.3/get-started/licensing/overview/#self-managed

Checks

  • New feature
  • Content gap
  • Support Follow-up
  • Small fix (typos, links, copyedits, etc)

@netlify
Copy link

netlify bot commented Nov 26, 2024

Deploy Preview for redpanda-docs-preview ready!

Name Link
🔨 Latest commit d84a071
🔍 Latest deploy log https://app.netlify.com/sites/redpanda-docs-preview/deploys/674da4b7d296f00008dce35e
😎 Deploy Preview https://deploy-preview-878--redpanda-docs-preview.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 site configuration.

@JakeSCahill JakeSCahill requested a review from oleiman November 28, 2024 09:25
Copy link
Member

@oleiman oleiman left a comment

Choose a reason for hiding this comment

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

lgtm

| None
| No change.

| xref:manage:security/authentication.adoc#kerberos[Kerberos authentication]
Copy link
Contributor

Choose a reason for hiding this comment

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

Just a style question. Should all links be title case, there seems to be a mix.

Copy link
Contributor

@asimms41 asimms41 left a comment

Choose a reason for hiding this comment

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

LGTM to a style question.

@JakeSCahill JakeSCahill merged commit 254a2fd into v-WIP/24.3 Dec 2, 2024
6 of 7 checks passed
@JakeSCahill JakeSCahill deleted the license-update branch December 2, 2024 12:26
Deflaimun pushed a commit that referenced this pull request Dec 2, 2024
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.

5 participants