Skip to content

Update the Views section on the CSRF Middleware for .csrf_field - #206

Closed
vendion wants to merge 2 commits into
goravel:masterfrom
vendion:f-csrf_field
Closed

vendion wants to merge 2 commits into
goravel:masterfrom
vendion:f-csrf_field

Conversation

@vendion

@vendion vendion commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

📑 Description

Update the CSRF Middleware section of the Views page to document the usage of the introduced .csrf_field template function, which returns a full HTML input field.

✅ Checks

  • Added test cases for my code

Update the CSRF Middleware section of the Views page to document the
usage of the introduced `.csrf_field` template function which returns a
full HTML input field.

Signed-off-by: Adam Jimerson <vendion@gmail.com>

@hwbrzzl hwbrzzl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks, could you update the zh file as well?

@vendion

vendion commented Oct 21, 2026

Copy link
Copy Markdown
Contributor Author

Yeah, I can. I thought that was still automated; that is why I didn't update those manually.

Signed-off-by: Adam Jimerson <vendion@gmail.com>
@vendion vendion closed this Oct 23, 2026
@vendion
vendion deleted the f-csrf_field branch October 23, 2026 14:11
@vendion
vendion restored the f-csrf_field branch October 23, 2026 14:11
@vendion vendion reopened this Oct 23, 2026

@hwbrzzl hwbrzzl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

So sorry, I forgot to tell you one thing: the new feature doc should be merged into upgrade/v1.19.0.

@vendion

vendion commented Oct 24, 2026

Copy link
Copy Markdown
Contributor Author

Sure, I'll rebase it over to that branch

@vendion vendion closed this Oct 24, 2026
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