-
Notifications
You must be signed in to change notification settings - Fork 28
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
Error when I try to produce "Inventory of Missing Data" inventory product #681
Comments
@Patowhiz : This module is a contribution of our colleague from Lesotho @Ralenkoane (Victor), using procedures. It has some issues and need further improvement. Not sure you have all the procedures required for this products. |
@mhabimana thanks for the response. Do you have a sample output that I can have a look at? |
Hi Marcellin,
What error does it throw. Guess it is something that I can fix without
delays.
Rg
…On Thu, Feb 10, 2022, 16:39 Marcellin ***@***.***> wrote:
@Patowhiz <https://github.com/Patowhiz> : This module is a contribution
of our colleague from Lesotho @Ralenkoane <https://github.com/Ralenkoane>
(Victor), using procedures. It has some issues and need further improvement.
—
Reply to this email directly, view it on GitHub
<#681 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADKHGWIEUWXPKAOVG6KQ353U2PE2JANCNFSM5OA2QQBA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@Patowhiz : It is working on my side: see below the output I am getting,but still it is not working well as desired because if you select many elements for example,it does not give you a good result. |
@Ralenkoane: Good to hear from you, long time. |
@Ralenkoane - Not sure you saw and read @Patowhiz message on top,he is complaining not getting desired products but I guest he is missing one of the procedures in his database. |
@mhabimana thanks for the sample output. I guess I don't have the procedures. Can you share them to me, would be useful to me for testing the code. I have the latest changes from the master. |
@Patowhiz - Sure. |
@Patowhiz : Below are the procedures,you need to run each of them once to create required tables in your database,you can read the scripts, they are self explanatory. Let me know how you are getting on. I am sure @Ralenkoane will be happy to respond to any other of your questions on this. |
@mhabimana and @Patowhiz this product was done through stored procedures, The development contributor (Victor, from Lesotho) was an expert in stored procedures and we allowed him use them. Note that we have been including these procedures in the installation and upgrade script files. If you run them the procedures will be created in your database. I have attached the upgrade script file in case you don't have it. |
@smachua @mhabimana I'm getting an error when @dannyparsons and I try to produce the "Inventory of Missing Data" inventory product. See screenshot below.
Looking at the code I noticed that this product is supposed to be generated by a procedure in the database level? So I presume my database script didn't have this procedure and current users have it? I wonder whether it would be useful to have this product done at the application level.
Thanks.
Error
Code
The text was updated successfully, but these errors were encountered: