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

Mobile v15 metadata component #196

Merged
merged 13 commits into from
Nov 21, 2024
Merged

Conversation

lauty95
Copy link
Collaborator

@lauty95 lauty95 commented Oct 9, 2024

Related issue #161

Doubts:
Source and Receiving are mocked from stonecrop, is it ok?

"Components" and "Quantity" are fixed, if I use component's slot from BeamMetadata component, I should move "ControlButtons" from Transfer, something like:

<ControlButtons
    :onCreate="create"
    :onSubmit="() => store.submit<StockEntry>('Stock Entry', stockEntryId)"
    :onCancel="() => store.cancel<StockEntry>('Stock Entry', stockEntryId)" />

<BeamMetadata :order="workOrder">
    <template #components>
        <ListView :items="items" />
    </template>
</BeamMetadata>

Captura de pantalla 2024-10-09 a la(s) 10 53 34 a  m

@lauty95 lauty95 requested a review from agritheory October 9, 2024 14:00
@lauty95 lauty95 self-assigned this Oct 9, 2024
@agritheory
Copy link
Owner

@lauty95 Can you rebase this to mobile_V15_se_workflow branch and incorporate the appropriate components from agritheory/stonecrop#171? Let @crabinak know if you need a hand

Copy link

github-actions bot commented Nov 5, 2024

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py10100% 
beam
   barcodes.py311358%18, 27, 29, 32, 43–44, 46–47, 52–54, 58–59
   handling_unit.py813754%25, 31, 63–65, 83, 94–95, 101–102, 104–105, 107–108, 110–111, 113, 115–118, 120–122, 124, 126, 128–129, 134, 136, 140, 142–143, 146, 148–149, 157
   printing.py61610%4–6, 8, 11–12, 15–16, 18–21, 24–25, 34–45, 47–54, 58–59, 61–62, 71–75, 78–79, 85–87, 90–91, 94–95, 97–99, 101–102, 104, 109, 111–112
beam/demand
   demand.py3142791%50, 65, 119, 204, 238, 282, 285, 454–455, 474–476, 481–482, 484, 486–488, 490, 534, 608, 612, 649, 655, 662, 683–684
   receiving.py108496%74, 126, 167, 206
   sqlite.py59198%40
   utils.py51590%55, 66, 72–74
beam/doctype/beam_mobile_route
   beam_mobile_route.py30100% 
beam/doctype/beam_settings
   beam_settings.py271737%10–12, 15–16, 18, 25, 30–33, 41–43, 47–48, 50
beam/doctype/handling_unit
   __init__.py9366%12–14
   handling_unit.py140100% 
beam/doctype/warehouse_types
   warehouse_types.py30100% 
beam/overrides
   sales_order.py10640%11–16
   stock_entry.py671774%61–62, 67–74, 80–81, 97, 112, 129, 132–133
   work_order.py10280%14, 16
beam/report/demand_map
   demand_map.py34340%4–5, 7–8, 11–12, 15–16, 103–110, 126, 128–142, 145–146
beam/report/handling_unit_traceability
   handling_unit_traceability.py44440%4–6, 9–11, 30–32, 34–35, 37–38, 41–42, 45–46, 48–49, 51–52, 54–55, 57, 63–65, 67–69, 71, 74–77, 79–81, 83–86, 89–90
beam/scan
   __init__.py1241984%20, 24–25, 30, 38, 70, 117–118, 128, 130–131, 135, 139–140, 143, 173, 202, 204, 222
   config.py26260%4, 6, 9–17, 19–21, 23–26, 28–30, 32–35, 37
www
   demand.py660%4, 6, 9–12
www/beam
   index.py770%4, 6, 9–13
TOTAL109032969% 

@lauty95
Copy link
Collaborator Author

lauty95 commented Nov 20, 2024

Receive context:
Added supplier and warehouse to description

Captura de pantalla 2024-11-20 a la(s) 10 05 00 a  m

Ship context:
Added warehouse and customer when it comes from a sale order

Captura de pantalla 2024-11-20 a la(s) 10 07 35 a  m

Transfer context: WIP

I worked in sqlite to add customer and supplier fields
On ship context change onMounted to useInfiniteScroll

@lauty95 lauty95 marked this pull request as ready for review November 20, 2024 13:21
@agritheory agritheory merged commit 5c9e99f into mobile_v15 Nov 21, 2024
5 checks passed
@agritheory agritheory deleted the mobile_v15-metadata-component branch November 21, 2024 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants