-
Notifications
You must be signed in to change notification settings - Fork 9
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
Implemented Item unit of measure component #140
Conversation
mkumbobeaty
commented
Mar 15, 2019
- closes I should be able to search for unit of measures I should be able to search for unit of measures #105
- closes I should be able to edit unit of measure I should be able to edit unit of measure #106
- closes I should be able to refresh list of unit of measures I should be able to refresh list of unit of measures #107
- closes I should be able to archive unit of measure I should be able to archive unit of measure #108
- closes I should be able to paginating list of unit of measures I should be able to paginating list of unit of measures #103
- closes I should be able to create unit of measure I should be able to create unit of measure #102
- closes I should be able to view list of unit of measures I should be able to view list of unit of measures #101
…to feature/uom
add ellipsize description and on hover display full text
src/layouts/BaseLayout.js
Outdated
title: 'List of available warehouses', | ||
'/resources/itemunit': { | ||
name: 'Item Unit', | ||
title: 'List of available Item unit of measure', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this be List of available units of measure for items
src/layouts/BaseLayout.js
Outdated
'/resources/warehouses': { | ||
name: 'Warehouses', | ||
title: 'List of available warehouses', | ||
'/resources/itemunit': { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be `unitsofmeasure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update all appearances of functions and variables names from "UnitOfMeasure to ItemUnit" and "ItemUnitOfMeasure to ItemUnit" for consistency
Use Item Unit on component level and Unit of Measure on UI level (what a user sees).
…to feature/uom
…into feature/uom