Dynamically rendering sales table data

[KishoreMenda]

contributors - @Gajjela-Koushik-Reddy @arshiya19

[hardikpatil]

I believe, you already pushed this commit in frontend 3 hours ago in #125 before creating this PR so I guess should close this PR

[KishoreMenda]

I made some more changes on top of existing code in sale.tsx. So I created a PR for that. Thanks.

…

On Tue, Apr 11, 2023, 3:45 PM Hardik Patil ***@***.***> wrote: I believe, you already have push this commit in frontend 3 hours ago — Reply to this email directly, view it on GitHub <#133 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AWY5ZX3DXIF7JJKKYOATEVTXAXNI7ANCNFSM6AAAAAAW2YBML4> . You are receiving this because you were assigned.Message ID: ***@***.***>

[hardikpatil]

I made some more changes on top of existing code in sale.tsx. So I created a PR for that. Thanks.
…
On Tue, Apr 11, 2023, 3:45 PM Hardik Patil @.> wrote: I believe, you already have push this commit in frontend 3 hours ago — Reply to this email directly, view it on GitHub <#133 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AWY5ZX3DXIF7JJKKYOATEVTXAXNI7ANCNFSM6AAAAAAW2YBML4 . You are receiving this because you were assigned.Message ID: @.>

PR Looks good to me, before pushing, just make sure whether your local frontend is update with your feature_branch. If not, do following:

1. git checkout frontend
2. git fetch origin
3. git pull origin frontend
4. git checkout your-feature-branch
5. git merge frontend
6. git push origin your-feature-branch

[DevangRaval1]

Hey, if you guys can check Issue #102 we have already started working on it.

[AbhinavReddy-Dev]

Hey, if you guys can check Issue #102 we have already started working on it.

@KishoreMenda @Gajjela-Koushik-Reddy @arshiya19 Please coordinate with @DevangRaval1 on how to take this forward as they have created an issue before this, I believe they do have some code to push. Again, please check the issues before jumping in to work on anything necessary so that the work is not duplicated and wasted in the end.

[skrlett]

Okay, sure. Thanks for letting us know.On Apr 11, 2023 19:33, Abhinav Reddy ***@***.***> wrote: Hey, if you guys can check Issue #102 we have already started working on it. @KishoreMenda @Gajjela-Koushik-Reddy @arshiya19 Please coordinate with @DevangRaval1 on how to take this forward as they have created an issue before this, I believe they do have some code to push. Again, please check the issues before jumping in to work on anything necessary so that the work is not duplicated and wasted in the end. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

For now, holding off approval on this PR since two team worked on same issue without each other knowledge.