search

bohua / nprinting-sense-on-demand

NPrinting On-demand extension for Qlik Sense
MIT License
10 stars 13 forks source link

Fails in a mashup #10

Closed celindho closed 6 years ago

celindho commented 6 years ago

If the extension is added to a mashup, it does not work. Apparently the websocket is opened to the address of the extension and not the app.

bohua commented 6 years ago

HI Christian,

Sorry for late reply, it sounds weird coz Nprinting extension is not using any websocket to communicate to NP server. The only only change to involve websocket is to collect current selections. I will have a look into this! Probably after new year since it's a busy period. :) Please feel free to contact me if things get urgent for you.

Brs, //Bohua

2017-12-15 15:21 GMT+01:00 Christian Lindholm notifications@github.com:

If the extension is added to a mashup, it does not work. Apparently the websocket is opened to the address of the extension and not the app.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/bohua/nprinting-sense-on-demand/issues/10, or mute the thread https://github.com/notifications/unsubscribe-auth/ADQoLxHmyb2uPjvPTODb0LvaKoS38Btzks5tAoCGgaJpZM4RDivu .

celindho commented 6 years ago

Hi,

Thanks for the reply! Based on my debugging effort, it appears to crash exactly when fetching the current selections. We have managed to figure out a workaround that should reduce the urgency of this problem but does not help with the original issue of getting it to work inside the mashup. I would none the less still appreciate any effort in looking into this bug :)

Br -Christian

safaGued commented 6 years ago

Hello @celindho @bohua ,

I have the same problem when using the mashup. The extension is working in the QS app in the hub but not with the mashup. @celindho : Could you please tell me what was your workaround for this issue? I must find an urgent solution

Thank you in advance Regards

celindho commented 6 years ago

@safaGued, in the mashup i added a link to the main document to a sheet where the button exists. If you make sure that the link opens in a new tab, the current selection will still ve active. The nprinting button works this way.

safaGued commented 6 years ago

Thank you a lot for your quick answer. I will try your solution and come back to you ;)

sgandotr commented 6 years ago

@celindho where are you adding the link? Are you opening an app in another tab?

bohua commented 6 years ago

Hi @celindho @safaGued

With the latest fix with Capability API, the problem should be resolved in mashups, since the selections are not pending on internal Qlik modules any more!

celindho commented 6 years ago

Thanks for the info. I'll test this out next time I have the chance :)