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.

Sign up for GitHub

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

Jump to bottom

Stock Changes Report is broken #6154

Closed

jmcameron opened this issue Dec 9, 2021 · 1 comment · Fixed by #6239

Assignees

Labels

Bug Report Stock

Collaborator

jmcameron commented Dec 9, 2021 •

edited

Loading

Stock > Report > Stock Changes Report

BUG! This report crashes for me with the server-side error message:

no such file or directory, open /Bhima/node_modules/chart.js/dist/Chart.min.js'
In the node_modules directory, the file name is chart.min.js (lower case)
Fixing this case name in server/controllers/stock/reports/stock_movement_report.handlebars does helps but there is still an error: TypeError: Cannot read property 'toLowerCase' of undefined

jmcameron added Bug Report Stock labels

jniles self-assigned this

Contributor

jniles commented Jan 4, 2022

It looks like the depot is required and that is the only bug here.

jniles mentioned this issue

fix(reports): require depots for stock changes #6239

Merged

bors bot added a commit that referenced this issue


          Merge #6239

ab8997f

6239: fix(reports): require depots for stock changes r=jniles a=jniles

Makes the depot field a required field for viewing the stock changes report.

Closes #6154.

![image](https://user-images.githubusercontent.com/896472/148118962-873611eb-465f-462f-a346-2bb634c2d405.png)


Co-authored-by: Jonathan Niles <jonathanwniles@gmail.com>

bors bot closed this as completed in

5f79ffb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment