Skip to content

Conversation

@junhongwang418
Copy link
Contributor

recipient, cc, and feature are just passed to the callback of onSubmit. Since we require onSubmit, the client already has access to recipient, cc, feature.

@netlify
Copy link

netlify bot commented Aug 25, 2022

Deploy Preview for stately-chebakia-3a1e20 ready!

Name Link
🔨 Latest commit 9a17de0
🔍 Latest deploy log https://app.netlify.com/sites/stately-chebakia-3a1e20/deploys/6307c90021ef67000853c1b5
😎 Deploy Preview https://deploy-preview-1059--stately-chebakia-3a1e20.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions
Copy link
Contributor

Released prerelease version 7.7.1-one-remove-unnecessary-props-2eba7c8.0.
You may now run npm install @appfolio/react-gears@one-remove-unnecessary-props

@JeremyRH
Copy link
Contributor

This is a major breaking change. There are many usages of these props and removing them will require many changes in apm_bundle: https://cs.github.com/?scopeName=All+repos&scope=&q=repo%3Aappfolio%2Fapm_bundle+%3CFeedBackButton

If you are willing to fix all the cases where these props are used, we can merge this.

@steventhan steventhan force-pushed the master branch 5 times, most recently from fc7b9c2 to 440bf24 Compare April 7, 2023 21:55
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.

4 participants