-
Notifications
You must be signed in to change notification settings - Fork 54
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
(fix) : fix the non display of the approved or completed table #55
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.
Nice work with the cleanup. Left a couple of more comments
className={styles.statusContainer} | ||
style={{ color: `${getStatusColor(entry.fulfillerStatus)}` }} |
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.
Are there any extra styles in the statusContainer
class? trying to access if it's still relevant after this refactor
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.
Looks like this is really relevant since it provides for the background of the table
"pages": [ | ||
{ | ||
"component": "root", | ||
"route": "laboratory" | ||
} | ||
], | ||
"extensions": [ | ||
{ |
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.
Same applies to this json file. 2 spaces
208ec26
to
5d2f804
Compare
6d7ba9a
to
9b5df73
Compare
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.
This PR is getting bigger so we'll make further refactors in follow up PRs
Requirements
Summary
This PR fixes the completed/ Approved table to display and also handles removal of unneccessay unused code.
Screenshots
Related Issue
Other