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

Sales Report View #78 #92

Merged
merged 7 commits into from
Apr 3, 2023
Merged

Sales Report View #78 #92

merged 7 commits into from
Apr 3, 2023

Conversation

hardikpatil
Copy link
Contributor

In this PR:

@hardikpatil hardikpatil added feature New feature or request UI Concerns UI presentation labels Apr 3, 2023
@hardikpatil hardikpatil self-assigned this Apr 3, 2023
kushalramakanth
kushalramakanth approved these changes Apr 3, 2023
View reviewed changes
Copy link
Contributor

@kushalramakanth kushalramakanth left a comment

Choose a reason for hiding this comment

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

LGTM! Working as expected

@DevangRaval1
Copy link
Contributor

ReviewHp

Looks good

@hardikpatil hardikpatil merged commit 50fdb24 into frontend Apr 3, 2023
@hardikpatil hardikpatil deleted the feature/sales-report-static branch April 3, 2023 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kushalramakanth kushalramakanth kushalramakanth approved these changes

@DevangRaval1 DevangRaval1 DevangRaval1 approved these changes

@AbhinavReddy-Dev AbhinavReddy-Dev Awaiting requested review from AbhinavReddy-Dev

@prestonmasseyblake prestonmasseyblake Awaiting requested review from prestonmasseyblake

@parthpandey1 parthpandey1 Awaiting requested review from parthpandey1

@prasannarajezzzy prasannarajezzzy Awaiting requested review from prasannarajezzzy

@abhilashSreenivasa abhilashSreenivasa Awaiting requested review from abhilashSreenivasa

Assignees

@hardikpatil hardikpatil

Labels
feature New feature or request UI Concerns UI presentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hardikpatil @DevangRaval1 @kushalramakanth