-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Standard Table #23
Standard Table #23
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Similar #12 |
Quality Gate passedIssues Measures |
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.
Changes Reviewed XD
@joaorodrs prop |
Description
Creating one table for each entity is kind of a code duplication issue, and this issue was highlighted by the code inspector the enterprise I'm working in uses. 60% of the code duplications issues came from this tables, mainly because of the stacked duplications of pages, search terms and filters states, that are the same thing for every table.
Closes #24 and #12
Type of change
How Has This Been Tested?
Just used the new "Standard Table" component in the existing users page and tested the listing and manipulation of the items in the table, and seems to work just fine.
Did you test this update on all browsers?
Checklist: