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
[PortaFly] Removing unused features from Account > Index #1977
[PortaFly] Removing unused features from Account > Index #1977
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actions column is also to be removed I think.
Not sure if that’s true |
I think we should not remove it. The actions column will apply to the given record and won't be triggering a batch update as how it was meant with bulk actions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AccountsTable.text.tsx
still has many things from actions and selections (imports, mocks, setup func, etc)
189148e
to
beb3322
Compare
4784fd1
to
cf94756
Compare
beb3322
to
4a7d857
Compare
4a7d857
to
29cf200
Compare
What this PR does / why we need it:
Removes features not needed in our first version of the account index.
#1985 covers the missing tests.
Which issue(s) this PR fixes
Fixes: https://issues.redhat.com/browse/THREESCALE-5430