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.

Sign up for GitHub

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

Jump to bottom

feat: Initial docs for DAL EnumField #1656

Merged
merged 4 commits into from
Jan 21, 2025
Merged

feat: Initial docs for DAL EnumField #1656

merged 4 commits into from
Jan 21, 2025
+1,160 −998

Conversation

dgrothaus-sw
Copy link
Contributor

Add documentation for the new DAL field type EnumField.

@dgrothaus-sw dgrothaus-sw added the Core @ct-core label Jan 17, 2025
@dgrothaus-sw dgrothaus-sw self-assigned this Jan 17, 2025
@dgrothaus-sw
Copy link
Contributor Author

I'll create this as a draft until a proper place to put the new type of documentation into is found.
Until then, the content can already be reviewed.

@dgrothaus-sw dgrothaus-sw mentioned this pull request Jan 17, 2025
Merged
@dgrothaus-sw dgrothaus-sw force-pushed the initial-docs-for-enum-field-in-dal branch from 185dc64 to 744d71b Compare January 17, 2025 09:24
mitelg
mitelg reviewed Jan 17, 2025
View reviewed changes
.devenv/nix-eval-cache.db Outdated Show resolved Hide resolved
node_modules/.yarn-integrity Outdated Show resolved Hide resolved
resources/references/core-reference/dal-reference/fields-reference/enum-field.md Outdated Show resolved Hide resolved
resources/references/core-reference/dal-reference/fields-reference/index.md Outdated Show resolved Hide resolved
yarn.lock Outdated Show resolved Hide resolved
@dgrothaus-sw
feat: Initial docs for DAL EnumField
09d9763 
Add documentation for the new DAL field type EnumField.
@dgrothaus-sw
Fix wording for title
d589aae
@dgrothaus-sw dgrothaus-sw force-pushed the initial-docs-for-enum-field-in-dal branch from 744d71b to d589aae Compare January 17, 2025 11:15
@Isengo1989 Isengo1989 marked this pull request as ready for review January 21, 2025 09:45
@Isengo1989 Isengo1989 merged commit 774929d into main Jan 21, 2025
8 checks passed
@Isengo1989 Isengo1989 deleted the initial-docs-for-enum-field-in-dal branch January 21, 2025 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Isengo1989 Isengo1989 Isengo1989 left review comments

@keulinho keulinho keulinho approved these changes

@mitelg mitelg mitelg approved these changes

Assignees

@dgrothaus-sw dgrothaus-sw

Labels
Core @ct-core
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dgrothaus-sw @mitelg @Isengo1989 @keulinho