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.

Sign up for GitHub

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

Jump to bottom

Summary Page API Integration #152

Open
decoles opened this issue Apr 14, 2023 · 4 comments
Open

Summary Page API Integration #152

decoles opened this issue Apr 14, 2023 · 4 comments
Assignees
@decoles @hamsaraj7106 @abhilashSreenivasa
Labels
api-integration Work with the backend team for API integration. backend concerns backend/API presentation UI Concerns UI presentation Urgent Need everyone's attention

Comments

@decoles
Copy link
Contributor

decoles commented Apr 14, 2023

Summary page is ready for API integration
@abhilashSreenivasa abhilashSreenivasa added UI Concerns UI presentation api-integration Work with the backend team for API integration. labels Apr 14, 2023
@abhilashSreenivasa
Copy link
Contributor

This is the summary page, and we will be making api calls for tables and charts.
Screen Shot 2023-04-14 at 1 59 15 PM
Screen Shot 2023-04-14 at 1 59 20 PM

@abhilashSreenivasa
Copy link
Contributor

Hi (to whoever decides to join us from the backend), I think the table API integration shouldn't be too complicated, But we are not completely sure about the charts. We need to understand the ReportType interface and the classes that are implementing them so that we can decide what exactly our chart is representing. Looking forward to working with you guys.

@AbhinavReddy-Dev AbhinavReddy-Dev added the backend concerns backend/API presentation label Apr 14, 2023
@hamsaraj7106 hamsaraj7106 self-assigned this Apr 14, 2023
@hamsaraj7106
Copy link
Contributor

Hi @abhilashSreenivasa,

Let's work on it.

@briswells
Copy link
Contributor

Some of the API routes for this are already being worked on for other issues. Stock and Feed.

Stock is a little different than how the screenshots show it. We did it that was so similar items with different expiration dates wouldn't take up multiple rows. Feed at this point is basically checks for anything that will expire soon. It will likely get renamed in the future but it should work fine for "soon to expire".

This was referenced Apr 16, 2023
Closed
Merged
hamsaraj7106 added a commit that referenced this issue Apr 17, 2023
@hamsaraj7106
Get api summary (#159)
9ff8a1c 
This PR address #152 
`/purchases`
`/currentstock`
`/wastemanagement`
`/soontoexpire`
@kbuffardi kbuffardi added the Urgent Need everyone's attention label Apr 25, 2023
@jayrevolinskyjr jayrevolinskyjr mentioned this issue May 2, 2023
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@decoles decoles

@hamsaraj7106 hamsaraj7106

@abhilashSreenivasa abhilashSreenivasa

Labels
api-integration Work with the backend team for API integration. backend concerns backend/API presentation UI Concerns UI presentation Urgent Need everyone's attention
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@kbuffardi @decoles @briswells @AbhinavReddy-Dev @hamsaraj7106 @abhilashSreenivasa