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

fix(docs): Updated development documentation. #55

Merged
merged 1 commit into from
May 23, 2023

Conversation

dlabaj
Copy link
Contributor

@dlabaj dlabaj commented May 23, 2023

Updated development documentation.

https://react-user-feedback.surge.sh

Closes issue #50

@dlabaj dlabaj added bug Something isn't working draft documentation Improvements or additions to documentation and removed draft labels May 23, 2023
@dlabaj dlabaj self-assigned this May 23, 2023
@dlabaj dlabaj added this to the 2023.05 - 5/31 milestone May 23, 2023
@dlabaj dlabaj linked an issue May 23, 2023 that may be closed by this pull request
Copy link
Contributor

@jessiehuff jessiehuff left a comment

Choose a reason for hiding this comment

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

LGTM! :)

Copy link
Contributor

@edonehoo edonehoo left a comment

Choose a reason for hiding this comment

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

Some phrasing adjustments to align with the React documentation I've done for components. Let me know if anything is incorrect or unclear!

packages/module/patternfly-docs/content/examples/basic.md Outdated Show resolved Hide resolved
packages/module/patternfly-docs/content/examples/basic.md Outdated Show resolved Hide resolved
packages/module/patternfly-docs/content/examples/basic.md Outdated Show resolved Hide resolved
packages/module/patternfly-docs/content/examples/basic.md Outdated Show resolved Hide resolved
packages/module/patternfly-docs/content/examples/basic.md Outdated Show resolved Hide resolved
packages/module/patternfly-docs/content/examples/basic.md Outdated Show resolved Hide resolved
packages/module/patternfly-docs/content/examples/basic.md Outdated Show resolved Hide resolved
packages/module/patternfly-docs/content/examples/basic.md Outdated Show resolved Hide resolved
packages/module/patternfly-docs/content/examples/basic.md Outdated Show resolved Hide resolved
packages/module/patternfly-docs/content/examples/basic.md Outdated Show resolved Hide resolved
chore(docs): updated examples with explainations.

chore(docs): updated examples with explainations.

chore(docs): Added async example.

chore(docs): reverted yarn.lock file.

chore(docs): Updated with fake forms.

chore(docs): Updated with fake forms.

chore(docs): Updated with review comments.

Review updates.

Fixed bad link.
@edonehoo
Copy link
Contributor

also - I don't see the "passes in email" example actually autofilling the test email. Is it meant to be displayed in the example form, or is it just happening in the background?

image (I was expecting this to have the test email already entered, but maybe I misunderstood)

@dlabaj dlabaj merged commit 1a6af52 into patternfly:v4 May 23, 2023
@github-actions
Copy link

🎉 This PR is included in version 1.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

dlabaj added a commit to dlabaj/react-user-feedback that referenced this pull request May 30, 2023
chore(docs): updated examples with explainations.

chore(docs): updated examples with explainations.

chore(docs): Added async example.

chore(docs): reverted yarn.lock file.

chore(docs): Updated with fake forms.

chore(docs): Updated with fake forms.

chore(docs): Updated with review comments.

Review updates.

Fixed bad link.
dlabaj added a commit to dlabaj/react-user-feedback that referenced this pull request May 31, 2023
chore(docs): updated examples with explainations.

chore(docs): updated examples with explainations.

chore(docs): Added async example.

chore(docs): reverted yarn.lock file.

chore(docs): Updated with fake forms.

chore(docs): Updated with fake forms.

chore(docs): Updated with review comments.

Review updates.

Fixed bad link.
dlabaj added a commit that referenced this pull request May 31, 2023
* fix(docs): Updated development documentation. (#55)

chore(docs): updated examples with explainations.

chore(docs): updated examples with explainations.

chore(docs): Added async example.

chore(docs): reverted yarn.lock file.

chore(docs): Updated with fake forms.

chore(docs): Updated with fake forms.

chore(docs): Updated with review comments.

Review updates.

Fixed bad link.

* fix(modal): Updated feedback modal to not use size as it's not needed for the PF v5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation released on @prerelease
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add support to allow asynchronous calls.
3 participants