Skip to content

Conversation

@matty0501
Copy link
Contributor

Context

Just something I noticed.

Summary

The snippet was incorrectly titled "Set Fields as Readonly On Edit", this PR renames it to "Remove Fields On Edit".

@matty0501 matty0501 requested a review from saifsultanc March 3, 2025 14:57
Copy link
Contributor

@saifsultanc saifsultanc left a comment

Choose a reason for hiding this comment

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

Thank you @matty0501 !

LGTM, just S&M. When you do, please remove the PR number from the end.

EDIT: I thought to do a quick walkthrough of how this would be: https://www.loom.com/share/9082be685ca24d05b349cc5d0553335b. Probably you are aware already, but just mentioning. LMK if you have any question!

@matty0501 matty0501 merged commit 97c5f3a into master Mar 5, 2025
3 of 4 checks passed
@matty0501 matty0501 deleted the matty0501-patch-3 branch March 5, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants