Skip to content

[EuiInMemoryTable] Added documentation for itemId prop and improved props table#4617

Merged
thompsongl merged 1 commit intoelastic:masterfrom
akashgp09:InMemoryTable
Mar 18, 2021
Merged

[EuiInMemoryTable] Added documentation for itemId prop and improved props table#4617
thompsongl merged 1 commit intoelastic:masterfrom
akashgp09:InMemoryTable

Conversation

@akashgp09
Copy link
Copy Markdown
Contributor

@akashgp09 akashgp09 commented Mar 5, 2021

Summary

This PR Fixes: #3381

Improved props table and added documentation for itemId prop.

Checklist

  • Check against all themes for compatibility in both light and dark modes
  • Checked in mobile
  • Checked in Chrome, Safari, Edge, and Firefox
    - [ ] Props have proper autodocs and playground toggles
  • Added documentation~~
    - [ ] Checked Code Sandbox works for the any docs examples
    - [ ] Added or updated jest tests
    - [ ] Checked for breaking changes and labeled appropriately
    - [ ] Checked for accessibility including keyboard-only and screenreader modes
    - [ ] A changelog entry exists and is marked appropriately

@kibanamachine
Copy link
Copy Markdown

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@thompsongl
Copy link
Copy Markdown
Contributor

jenkins test this

@thompsongl thompsongl added the skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation) label Mar 15, 2021
@kibanamachine
Copy link
Copy Markdown

Preview documentation changes for this PR: https://eui.elastic.co/pr_4617/

@thompsongl thompsongl merged commit aa53b95 into elastic:master Mar 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Use on PRs to skip changelog requirement (Don't delete - used for automation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[EuiInMemoryTable] Needs to document its itemId prop

3 participants