Skip to content

Add additional license clarifications - #698

Merged
JamesW1 merged 4 commits into
mainfrom
license-overview-updates
Jun 1, 2026
Merged

JamesW1 merged 4 commits into
mainfrom
license-overview-updates

Conversation

@JamesW1

@JamesW1 JamesW1 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor
  • Important note on PUBLIC_URL
  • Note on OIG key max activations
  • Emphasize importance of deactivating a license before destroying the instance it is bound to

@JamesW1
JamesW1 requested a review from a team as a code owner June 1, 2026 19:06
@vercel

vercel Bot commented Jun 1, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Error Error Jun 1, 2026 8:08pm

Request Review

@ComfortablyCoding ComfortablyCoding 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.

Some minor comments, not a huge fan of qualified URL as that is vague. No proposed alternative currently.

Comment thread content/licensing/1.overview.md Outdated
Comment thread content/licensing/1.overview.md Outdated
Comment thread content/licensing/1.overview.md Outdated
| `PROJECT_OWNER` | Registered owner and primary contact responsible for your Directus instance. | |


::callout{icon="material-symbols:warning-rounded" color="warning"}

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.

We should probably mirror #698 (comment) here

Co-authored-by: judda <44623501+ComfortablyCoding@users.noreply.github.com>
@JamesW1
JamesW1 merged commit dd23540 into main Jun 1, 2026
3 of 4 checks passed
@JamesW1
JamesW1 deleted the license-overview-updates branch June 1, 2026 20:08

This branch had an error being deployed

1 failed deployment
Preview — 1407c5e7 Deployed Jun 1, 2026 by vercel[bot]
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.

2 participants