Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix prevent submitted form data setting #117

Merged
merged 7 commits into from
Sep 9, 2022

Conversation

wazelin
Copy link
Member

@wazelin wazelin commented Feb 17, 2022

No description provided.

Co-authored-by: poyuki <37804293+poyuki@users.noreply.github.com>
@wazelin wazelin requested a review from poyuki February 17, 2022 14:51
Copy link
Contributor

@poyuki poyuki left a comment

Choose a reason for hiding this comment

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

  • New code is covered by tests (if applicable)
  • Tests are running successfully (old and new ones) on my local machine (if applicable)
  • New code is respecting code style rules
  • New code is respecting best practices
  • New code is not subject to concurrency issues (if applicable)
  • Feature is working correctly on my local machine (if applicable)
  • Acceptance criteria are respected
  • Pull request title and description are meaningful
  • Pull request's target is not master
  • Commits are following conventional commits
  • Changelog is updated according to changes (if applicable)
  • Documentation is updated according to changes (if applicable)

Copy link

@quintanilhar quintanilhar left a comment

Choose a reason for hiding this comment

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

  • New code is covered by tests
  • Tests are running successfully on my local machine
  • New code is respecting code style rules
  • New code is respecting best practices
  • Feature is working correctly on my local machine
  • Acceptance criteria are respected
  • Pull request title and description are meaningful
  • Pull request's target is not master
  • Commits are following conventional commits
  • Commits messages are meaningful
  • Commits are atomic
  • Changelog is updated according to changes
  • Documentation is updated according to changes

@wazelin wazelin merged commit 99c1e33 into develop Sep 9, 2022
@wazelin wazelin deleted the fix/prevent-submitted-form-data-setting branch September 9, 2022 14:59
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants