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

feat(earnings): Implement earnings tabular list UI #181

Merged

Conversation

Mloweedgar
Copy link
Contributor

@Mloweedgar Mloweedgar commented Oct 27, 2021

Description

Developed UI for listing earnings, currently dummy earnings are listed from mock API as we wait for the earnings API to return more well-structured data.

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added
  • Docs have been added / updated

Issue

What is the current behavior?
There was no earnings UI before

What is the new behavior?
Earnings tabular list UI with filters has been implemented

Other useful information

Only earnings table UI filled with dummy data is displayed, another PR will be done to address the integration of earnings list UI with earnings API.

ref #179

Copy link
Collaborator

@nmcharlton nmcharlton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm a little concerned by how specific the styling is to individual elements, but I appreciate that the UI design is different from existing tools.

Maybe we extract common elements from this tool once we start implementing the other tools based on the same design.

src/components/EarningsTable/EarningsTable.js Outdated Show resolved Hide resolved
src/components/EarningsTable/EarningsTable.js Outdated Show resolved Hide resolved
src/components/EarningsTable/EarningsTable.js Outdated Show resolved Hide resolved
src/components/EarningsTable/EarningsTable.js Outdated Show resolved Hide resolved
src/components/EarningsTable/EarningsTable.styles.js Outdated Show resolved Hide resolved
src/components/EarningsTable/EarningsTable.styles.js Outdated Show resolved Hide resolved
src/components/EarningsTable/EarningsTable.styles.js Outdated Show resolved Hide resolved
@Mloweedgar Mloweedgar closed this Nov 23, 2021
@Mloweedgar Mloweedgar reopened this Nov 23, 2021
@Mloweedgar Mloweedgar closed this Nov 24, 2021
@Mloweedgar Mloweedgar reopened this Nov 24, 2021
Copy link
Collaborator

@nmcharlton nmcharlton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good so far!

@Mloweedgar Mloweedgar requested a review from a team as a code owner December 1, 2021 20:02
@nmcharlton nmcharlton merged commit b7adb1c into Greenstand:master Dec 1, 2021
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.

3 participants