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

storage: Prepare (a bit) for full react #9991

Merged

Conversation

mvollmer
Copy link
Member

@mvollmer mvollmer commented Sep 5, 2018

This fixes issues found while looking into #9263

- Make sure all tables have a tbody.
- Add key properties that React warns about.
- Use Browser.set_input_text in the tests when setting input text elements.
As pointed out by React, thanks!

Closes cockpit-project#9991
@mvollmer mvollmer force-pushed the storage-prepare-for-full-react branch from 3b97477 to 229950b Compare September 5, 2018 10:56
@mareklibra mareklibra mentioned this pull request Sep 5, 2018
44 tasks
@mvollmer mvollmer changed the title storage: Rrepare (a bit) for full react storage: Prepare (a bit) for full react Sep 5, 2018
@martinpitt martinpitt merged commit 3e7bba0 into cockpit-project:master Sep 5, 2018
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.

None yet

2 participants