litzvi / avc-beta

trying to build first demo for avc
0 stars 0 forks source link

General Inventory History Report - amount and measure unit seperate #619

Closed litzvi closed 2 years ago

litzvi commented 2 years ago

she want's them in separate columns. so she can use it conveniently in excel. e.g. 500 | KG

mei309 commented 2 years ago

not possible. its totals with lbs and kg in some cases so which one i need to put? i dont think its a good change

litzvi commented 2 years ago

not possible. its totals with lbs and kg in some cases so which one i need to put? i dont think its a good change

why do you care, she asked. I can give it separate if you want.

litzvi commented 2 years ago

or only do it for excel export? what do you think?

mei309 commented 2 years ago

not possible. its totals with lbs and kg in some cases so which one i need to put? i dont think its a good change

why do you care, she asked. I can give it separate if you want.

so give it separate or give both and i will show it separately for excel only

litzvi commented 2 years ago

okay I added. only in "Inventory=>General Stock=> General Stock/oder"

mei309 commented 2 years ago

i think you got it wrong. she has a totally different problem. the cloumns are mixing up

litzvi commented 2 years ago

@david told me that. Anyway it's correct, so they can make calculations etc. In excel. Can you see what her problem is?

On Thu, Oct 14, 2021, 15:28 mei309 @.***> wrote:

Assigned #619 https://github.com/litzvi/avc-beta/issues/619 to @litzvi https://github.com/litzvi.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/litzvi/avc-beta/issues/619#event-5462155769, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMZHF3AJRK7TRO4FEQNLZ2TUG2ID3ANCNFSM5F4O57EA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

litzvi commented 2 years ago

because it isn't happening by me

mei309 commented 2 years ago

@david told me that. Anyway it's correct, so they can make calculations etc. In excel. Can you see what her problem is?

On Thu, Oct 14, 2021, 15:28 mei309 @.***> wrote:

Assigned #619 https://github.com/litzvi/avc-beta/issues/619 to @litzvi https://github.com/litzvi.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/litzvi/avc-beta/issues/619#event-5462155769, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMZHF3AJRK7TRO4FEQNLZ2TUG2ID3ANCNFSM5F4O57EA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I see the problem and I tried reproducing it all afternoon but didn't succeed. it's something with excel that it breaks after comma I will find it tonight

mei309 commented 2 years ago

@david told me that. Anyway it's correct, so they can make calculations etc. In excel. Can you see what her problem is? On Thu, Oct 14, 2021, 15:28 mei309 @.***> wrote: Assigned #619 <#619> to @litzvi https://github.com/litzvi. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#619 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMZHF3AJRK7TRO4FEQNLZ2TUG2ID3ANCNFSM5F4O57EA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

we should give to do the calculations by us. all sums is easy and we should add it so they dont need to use excel

mei309 commented 2 years ago

and you should check what report they want that way (i will try talking to david but he really doesn't understand me). this one she wrote only because of the Excel error

litzvi commented 2 years ago

No need to reinvent the wheel.. Excel is a powerful tool that they know.

On Fri, Oct 15, 2021, 06:25 mei309 @.***> wrote:

@david https://github.com/david told me that. Anyway it's correct, so they can make calculations etc. In excel. Can you see what her problem is? … <#m-5425338939302599264> On Thu, Oct 14, 2021, 15:28 mei309 @.***> wrote: Assigned #619 https://github.com/litzvi/avc-beta/issues/619 <#619 https://github.com/litzvi/avc-beta/issues/619> to @litzvi https://github.com/litzvi https://github.com/litzvi. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub <#619 (comment) https://github.com/litzvi/avc-beta/issues/619#event-5462155769>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMZHF3AJRK7TRO4FEQNLZ2TUG2ID3ANCNFSM5F4O57EA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .

we should give to do the calculations by us. all sums is easy and we should add it so they dont need to use excel

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/litzvi/avc-beta/issues/619#issuecomment-943836316, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMZHF3DB4G4R6EVLT77S53TUG5RGTANCNFSM5F4O57EA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

litzvi commented 2 years ago

and you should check what report they want that way (i will try talking to david but he really doesn't understand me). this one she wrote only because of the Excel error

He told me that Noonoo asked him that. So if there was a misunderstanding it's by him or he assumed that's the reason. To fix Noonoo's problem, isn't it about escaping the apostrophe when it's in the cell? Anyhow, it would be good to separate in the future for excel so they can easily create there own functions.

litzvi commented 2 years ago

Why didn't you do this before you upload?

mei309 commented 2 years ago

Why didn't you do this before you upload?

because it wasn't simple

mei309 commented 2 years ago

now done only in inventory=>general stock=>stock/ordered lets see if this is what they want and if they want to implement it on all weights (because if yes i will do it automatically, getting a bunch of extra data isnt correct)

litzvi commented 2 years ago

now done only in inventory=>general stock=>stock/ordered

this is the one they need. David asked me about it again, he said that's the way they are use to seeing it. You can listen to him and decide. I don't think it's important in the browser report.

mei309 commented 2 years ago

did also on the software

mei309 commented 2 years ago

it looks like this was the only place they wanted it. so closing