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

Inbound shipments: there is no 'Transport details' field on manual inbounds #5103

Closed
regotaina opened this issue Oct 14, 2024 · 5 comments
Closed
Labels
bug Something is borken Build Tested: None This issue cannot be build tested or will not be build tested Severity: Low Bugs that don't block workflows. UX like wrong fonts, bad layout

Comments

@regotaina
Copy link
Contributor

What went wrong? 😲

In Inbound Shipments, if you are using .APK and click on 'More' on the right side, you will not have the option to add Transport details, as per picture below:

Screenshot 2024-10-14 at 3 15 03 PM

If you are in the browser and click on 'More' on the right side, you will have the option to add Transport details, as per picture below:
Screenshot 2024-10-14 at 3 13 29 PM

Expected behaviour 🤔

Transport Details should be available in .APK

How to Reproduce 🔨

Steps to reproduce the behaviour:

  1. Go to 'Inbound Shipments'
  2. Click on 'More'
  3. Scroll down to 'Transport Details'
  4. See error - the option is not there

Your environment 🌱

  • Open mSupply Version: V2.3RC8
  • Legacy mSupply Central Server Version:
  • Platform: Android
@regotaina regotaina added bug Something is borken needs triage needs daily triage New bug to be triaged by the release team labels Oct 14, 2024
@regotaina regotaina changed the title Inbound shipments: there is no 'Transport details' field in .APK, just on brownser Inbound shipments: there is no 'Transport details' field in .APK, just on browser Oct 14, 2024
@mark-prins
Copy link
Collaborator

I've just tested on v2.2.02 and the details show in the apk.
Could it be that the tablet is on an older version @regotaina ?

@lache-melvin
Copy link
Contributor

I have transport details on RC8 APK.. 👀 However, you only see transport reference on an Inbound Shipment if it was created via transfer from another store, manual inbounds don't have it - is that what happened here? 🙏

@regotaina
Copy link
Contributor Author

@mark-prins I think Lache is right, it doesn't show up if it's manually created. But in theory this may still be recorded, right? Not sure if it's only valuable for the supplier store, not the customer one.
Def using RC8 here :)

I have transport details on RC8 APK.. 👀 However, you only see transport reference on an Inbound Shipment if it was created via transfer from another store, manual inbounds don't have it - is that what happened here? 🙏

@lache-melvin lache-melvin added Severity: Low Bugs that don't block workflows. UX like wrong fonts, bad layout and removed needs daily triage New bug to be triaged by the release team labels Oct 15, 2024
@lache-melvin
Copy link
Contributor

Will leave with triage team, if we want to show transport details section on manual inbound shipments

@lache-melvin lache-melvin changed the title Inbound shipments: there is no 'Transport details' field in .APK, just on browser Inbound shipments: there is no 'Transport details' field on manual inbounds Oct 15, 2024
@andreievg
Copy link
Collaborator

This is expected for now, will update docs: msupply-foundation/msupply_docs#114

For future we might want to add transport details to manual

@marthakeezy marthakeezy added the Build Tested: None This issue cannot be build tested or will not be build tested label Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is borken Build Tested: None This issue cannot be build tested or will not be build tested Severity: Low Bugs that don't block workflows. UX like wrong fonts, bad layout
Projects
None yet
Development

No branches or pull requests

6 participants