-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Access updates #4203
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
Access updates #4203
Conversation
…e messages contains wrong statement
… the one supplied in props
💥 No ChangesetLatest commit: ba2fc6f Merging this PR will not cause any packages to be released. If these changes should not cause updates to packages in this repo, this is fine 🙂 If these changes should be published to npm, you need to add a changeset. This PR includes no changesetsWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Click here to learn what changesets are, and how to add one. Click here if you're a maintainer who wants to add a changeset to this PR |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit ba2fc6f:
|
@TheHollidayInn There are merge conflicts that need to be resolved. |
Greetings @TheHollidayInn, Thank you for your efforts creating this example to go with #4161. Your efforts are much appreciated. Since the PR you created was in conjunction with #4161 and that PR was closed in favor of one that could updated, I will be closing this as well. Please note that I have taken most of the work you have provided here and merged it into #4390 with some minor changes, but all in all, the work you provided was very helpful. Your efforts are appreciated and look forward to moving forward with some long over due updates to aria live messaging. Please feel free to add any feedback to #4390 and if you see any other opportunities for feedback, please feel free to let me know there. Thank you again! |
While reviewing #4161, I thought it might be a good idea to add an example in the documentation. Let me know if this is wanted.
Also, I'm sorry, but I could not find radegran's repo when trying to assign the base repository. So, I just made the PR on to master. Please merge #4161 first, then the commits should clear up.