OrkestraOnline / Orkestra_Online

Orkestra Online is a cloud based platform for deploying, documenting and securing DynamoBIM and Grasshopper content.
https://www.orkestra.online
36 stars 2 forks source link

Package Missing Notification When Copying from Workspace to Workspace #186

Closed johnpierson closed 1 year ago

johnpierson commented 1 year ago

Describe the bug This notification can be useful, but I got a v. v.

To Reproduce Steps to reproduce the behavior:

  1. Copy DYN from one workspace (with package use) to another (without packages uploaded)

Expected behavior To describe which packages are missing

Screenshots image

In Which context did it happen (please select from the following list) : Orkestra Desktop app

MostafaElAyoubi commented 1 year ago

hey @johnpierson ! Thank you very much for reporting this 😃 . This dialog is indeed useful as it reports something that will definitely cause definitions to not work if the adjustments aren't made in the Workspace settings.

Two questions :

johnpierson commented 1 year ago

I can definitely share the definitions. The ones I believe are triggering it are from Revit 2023 back to 2022 and the iropython package. (Not needed before 2023)

On Mon, Dec 5, 2022 at 7:35 AM Mostafa El Ayoubi @.***> wrote:

hey @johnpierson https://github.com/johnpierson ! Thank you very much for reporting this 😃 . This dialog is indeed useful as it reports something that will definitely cause definitions to not work if the adjustments aren't made in the Workspace settings.

Two questions :

  • does this happen for all definitions?
  • In case this is reproducible with a specific definition, could you please share it? We'll look into it right away!

— Reply to this email directly, view it on GitHub https://github.com/MostafaElAyoubi/Orkestra_Online/issues/186#issuecomment-1337480447, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADYD5VFMNQK5CESAUFB7NXTWLX4TNANCNFSM6AAAAAASSJZMYY . You are receiving this because you were mentioned.Message ID: @.***>

MostafaElAyoubi commented 1 year ago

Thank you for the prompt response! Alright! That's super helpful. I'll try to reproduce the issue using this info and will get back to you if the definition is needed.

MostafaElAyoubi commented 1 year ago

Fixed in 1.7.1