Aung Ko Ko Lin (Quartile) comments

Results 152 comments of


                                            Aung Ko Ko Lin (Quartile)

[16.0][ADD] stock_valuation_specific_identification

@matt454357 I decided to go with `stock_valuation_fifo_lot` for two reasons: - The community is more likely to accept an existing module than a new one. - In my opinion, this...

[16.0][ADD] stock_valuation_specific_identification

> Anyway, I am happy to contribute to the `stock_valuation_fifo_lot` module. I would like to add more than just the revaluation stuff. For example, drop the `stock_no_negative` requirement, and enforce...

[16.0][ADD] stock_picking_location_check

@oca/stock-logistics-warehouse-maintainers Is is possible to take a look at this new module?

[16.0][FIX] excel_import_export: remove sudo() when importing record

@kittiu Can you please review this changes?

[16.0][FIX] excel_import_export: remove sudo() when importing record

@oca/server-tools-maintainers Can we get this PR merged?

[12.0][FIX] excel_import_excel: fix adding new row in excel for pre-defined row and add extend attribute

@kittiu Can you please review this PR?

[12.0][FIX] excel_import_excel: fix adding new row in excel for pre-defined row and add extend attribute

> What is this Travis error? @kanda999 I don't know what is the reason why travis failed? I think we need to change CI from travis to github in 12.0...

[12.0][FIX] excel_import_excel: fix adding new row in excel for pre-defined row and add extend attribute

@thomaspaulb Done!

[16.0][IMP] sale_tier_validation: add the option for blocking sale order print

Since the error displayed in my code is related to a business rule (i.e., the order needs approval before printing) and not a direct data validation issue, I changed from...

[16.0][IMP] sale_tier_validation: add the option for blocking sale order print

@StefanRijnhart Can we get this PR merged?