Skip to content
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

Update glossary.md #986

Merged
merged 1 commit into from
Oct 18, 2024
Merged

Update glossary.md #986

merged 1 commit into from
Oct 18, 2024

Conversation

sapetti9
Copy link
Contributor

Looks like this should be indented differently. @roll can you please confirm. Thanks!

@sapetti9 sapetti9 requested a review from roll October 16, 2024 16:05
Copy link

Deploying datapackage with  Cloudflare Pages  Cloudflare Pages

Latest commit: a790114
Status: ✅  Deploy successful!
Preview URL: https://2716462d.datapackage-6gp.pages.dev
Branch Preview URL: https://sapetti9-patch-1.datapackage-6gp.pages.dev

View logs

@roll
Copy link
Member

roll commented Oct 17, 2024

@sapetti9
Technically "Custom Properties" belongs to "Descriptor" so both heading levels are possible. @peterdesmet WDYT?

@peterdesmet
Copy link
Member

I think it makes the most sense as its own heading in the dropdown.

@roll roll merged commit cc38c41 into main Oct 18, 2024
2 checks passed
@roll roll deleted the sapetti9-patch-1 branch October 18, 2024 09:15
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.

3 participants