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.

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

Update 05_Simple_Contact_Form.md #416

Closed
wants to merge 39 commits into from
Closed

Conversation

erangaddd
Copy link
Contributor

No description provided.

GuySartorelli and others added 30 commits September 1, 2023 10:39
…rror_codes_changelog

DOC add errorpage allowed_error_codes to 5.2.0 changelog
…andom-password

DOC Create new users with random passwords
…ide-pagetypes-config

DOC Document new hide_pagetypes config
…eprecate

DOC Deprecated $having parameter
…elect-all

DOC Select all and deselect all files
…chema-exception

DOC LogicException when no react component
@erangaddd
Copy link
Contributor Author

  use SilverStripe\Forms\TextareaField;

@erangaddd erangaddd closed this Dec 5, 2023
@erangaddd erangaddd reopened this Dec 5, 2023
Copy link
Contributor Author

@erangaddd erangaddd left a comment

Choose a reason for hiding this comment

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

Added line
use SilverStripe\Forms\TextareaField;

@GuySartorelli
Copy link
Member

Thank you for submitting this.

As a matter of house keeping - in future please provide a clear description of what change you're making and why in the PR description, and adhere to our documented commit message guidelines.

There is a PHP linting issue in the CI - can you please resolve that?

@GuySartorelli
Copy link
Member

(oops sorry, didn't mean to close it)

@GuySartorelli
Copy link
Member

@erangaddd It looks like you've pulled through a bunch of commits from the 5 branch. Please reset your commits so that this can be merged.

@erangaddd erangaddd closed this Dec 14, 2023
@erangaddd erangaddd deleted the patch-1 branch December 14, 2023 21:51
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.

5 participants