Skip to content

Generating function component by default #1271

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

Merged
merged 12 commits into from
Jul 20, 2023

Conversation

ahangarha
Copy link
Collaborator

@ahangarha ahangarha commented May 1, 2023

Summary

This PR changes the functionality of component generation by creating function components.

Pull Request checklist

  • Add/update test to cover these changes
  • Update documentation
  • Update CHANGELOG file

Other Information

Closes #1067

Sorry, something went wrong.

@justin808 justin808 added this to the 3.0 milestone May 6, 2023
@ahangarha ahangarha force-pushed the add-option-for-generating-functional-component branch from c175289 to d341357 Compare May 16, 2023 11:43
@ahangarha ahangarha changed the title Add option for generating functional component Generating function component by default May 17, 2023
@ahangarha ahangarha force-pushed the add-option-for-generating-functional-component branch from 1d2bc96 to f755896 Compare June 11, 2023 18:48
@ahangarha ahangarha requested a review from justin808 June 11, 2023 18:52
@ahangarha ahangarha force-pushed the add-option-for-generating-functional-component branch from 45d1bbe to bd15add Compare July 20, 2023 21:40
@ahangarha ahangarha force-pushed the add-option-for-generating-functional-component branch from 3e1c346 to 7361cb8 Compare July 20, 2023 21:54
@ahangarha ahangarha requested a review from Judahmeek July 20, 2023 22:10
Copy link
Collaborator

@Judahmeek Judahmeek left a comment

Choose a reason for hiding this comment

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

One question on Changelog entry.

CHANGELOG.md Outdated
@@ -21,6 +21,9 @@ _Please add entries here for your pull requests that are not yet released._
#### Bug Fixes
- Fix ReactDomClient initialization error during SSR. #1278

#### Changed
- The `react:component` generator now generates a function component by default #1271

Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't this PR go under "unreleased" rather then under 2.7.1, which was released last month?

@Judahmeek Judahmeek merged commit f02d973 into master Jul 20, 2023
@Judahmeek Judahmeek deleted the add-option-for-generating-functional-component branch July 20, 2023 22:33
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.

Suggestion to add functional option to component generator
3 participants