Skip to content

Updates Custom Field Constructor Documentation#12892

Merged
mergify[bot] merged 1 commit into
playframework:mainfrom
RommelTJ:patch-1
Feb 18, 2025
Merged

Updates Custom Field Constructor Documentation#12892
mergify[bot] merged 1 commit into
playframework:mainfrom
RommelTJ:patch-1

Conversation

@RommelTJ
Copy link
Copy Markdown
Contributor

@RommelTJ RommelTJ commented Oct 3, 2024

Pull Request Checklist

Helpful things

Fixes

Fixes #5172

Purpose

Updates the Custom Field Constructor documentation to be more clear. Provides suggested filenames and directories so that a beginner can get started with creating custom field constructors.

If anything is still not clear, or we need further edits, let me know. I'm happy to make them.

Background Context

Outstanding issue with the documentation since October 2015. I created a minimal ("Hello Play") sample project with a minimal custom field constructor to verify the approach.

References

#5172

Addresses playframework#5172

Provides suggested filenames/directories to help implement custom field constructors
Copy link
Copy Markdown
Member

@mkurz mkurz left a comment

Choose a reason for hiding this comment

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

Thanks!

@mkurz
Copy link
Copy Markdown
Member

mkurz commented Feb 18, 2025

@Mergifyio backport 3.0.x 2.9.x

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Feb 18, 2025

backport 3.0.x 2.9.x

✅ Backports have been created

Details

@mergify mergify Bot merged commit ca8e8db into playframework:main Feb 18, 2025
mergify Bot added a commit that referenced this pull request Feb 18, 2025
[3.0.x] Updates Custom Field Constructor Documentation (backport #12892) by @RommelTJ
mergify Bot added a commit that referenced this pull request Feb 18, 2025
[2.9.x] Updates Custom Field Constructor Documentation (backport #12892) by @RommelTJ
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve field constructor documentation

2 participants