bromagosa / Snap4Arduino

Binding Snap! and Arduino together
http://snap4arduino.rocks
GNU Affero General Public License v3.0
132 stars 85 forks source link

Premature leaving page dialog #74

Closed stirlingstout closed 7 years ago

stirlingstout commented 7 years ago

Snap4Arduino! 0.1.5-beta

I've got a project which uses a Python server to implement a file server, so Snap! or Snap4Arduino can write/rread data (e.g., logged from an Arduino) to a file.

I've just been trying to implement a missing block (see the pictures) and when I try to Apply the change to the blocks in the first picture I get asked if I want to leave the page. If I take the call block out then Apply works OK. I've net yet confirmed if this happens in Snap!

capture6

capture1 capture4 capture5 capture2

bromagosa commented 7 years ago

Hi John!

Obvious question (sorry), have you tried using the latest Snap4Arduino version?

I assume you're using the desktop flavor, right? If that's the fact, you could actually implement file system operations without the need of an intermediary server, here's an example:

image

Still, I'm not sure what can be causing the leave page dialog to show up. Can you attach the project XML here so I can take a look?

Thanks! :)

stirlingstout commented 7 years ago

Bernat

I’d just downloaded the version that I thought was the latest from the site (1.0.6 Beta). I haven’t tried it with the web version yet.

I didn’t know about the require(‘fs’) technique, which is great, but I wanted a solution for ‘normal’ Snap! to run in the browser.

I’ll have another go at reproducing the problem and send you the project XML.

John

From: Bernat Romagosa [mailto:notifications@github.com] Sent: 27 September 2016 07:35 To: edutec/Snap4Arduino Cc: John Stout; Author Subject: Re: [edutec/Snap4Arduino] Premature leaving page dialog (#74)

Hi John!

Obvious question (sorry), have you tried using the latest Snap4Arduino version?

I assume you're using the desktop flavor, right? If that's the fact, you could actually implement file system operations without the need of an intermediary server, here's an example:

[image]https://cloud.githubusercontent.com/assets/1016697/18862527/007bf210-848d-11e6-8cf7-ee86bc26d829.png

Still, I'm not sure what can be causing the leave page dialog to show up. Can you attach the project XML here so I can take a look?

Thanks! :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/edutec/Snap4Arduino/issues/74#issuecomment-249778456, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACEABwmUw_xT6OqcweT_EpynZDUIDLJzks5quLkVgaJpZM4KHCM7.

This email was Virus checked by KGV Sophos UTM. itservices@kgv.ac.ukmailto:itservices@kgv.ac.uk

[King George V Logo] John Stout Management Information Systems Coordinator Tel: 01704 530601 E-Mail: JSS@kgv.ac.uk Web: www.kgv.ac.uk

Information contained in this e-mail is intended for the use of the addressee only and is confidential. Any dissemination, distribution, copying or use of this communication without prior permission of the addressee is strictly prohibited.

Every effort has been made to ensure that any attachment to this mail does not contain virus. King George V College has taken every reasonable precaution to minimise this risk, neither it nor the sender can accept liability for any damage which you sustain as a result of software viruses. You should carry out your own virus checks before opening any attachments.

[Think Before You Print]

jguille2 commented 7 years ago

Last version is 1.1.6 beta-rc3 released on 19/08/2016.

Current site is http://snap4arduino.org/

stirlingstout commented 7 years ago

The version I'm running I downloaded from the site http://snap4arduino.org/ yesterday, using the Microsoft Windows (64 bits) link, but clicking on the logo when running Snap4Arduino and selecting About Snap4Arduino... shows 0.1.5-beta.

John


From: Joan Guill?n i Pelegay notifications@github.com Sent: 27 September 2016 09:11 To: edutec/Snap4Arduino Cc: John Stout; Author Subject: Re: [edutec/Snap4Arduino] Premature leaving page dialog (#74)

Last version is 1.1.6 beta-rc3 released on 19/08/2016.

Current site is http://snap4arduino.org/

You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/edutec/Snap4Arduino/issues/74#issuecomment-249796365, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACEAB_z0OjxMP0UXoQ189tbXOpV4nhCPks5quM-wgaJpZM4KHCM7.

This email was Virus checked by KGV Sophos UTM. itservices@kgv.ac.uk

[King George V Logo] John Stout Management Information Systems Coordinator Tel: 01704 530601 E-Mail: JSS@kgv.ac.uk Web: www.kgv.ac.uk

Information contained in this e-mail is intended for the use of the addressee only and is confidential. Any dissemination, distribution, copying or use of this communication without prior permission of the addressee is strictly prohibited.

Every effort has been made to ensure that any attachment to this mail does not contain virus. King George V College has taken every reasonable precaution to minimise this risk, neither it nor the sender can accept liability for any damage which you sustain as a result of software viruses. You should carry out your own virus checks before opening any attachments.

[Think Before You Print]

bromagosa commented 7 years ago

Ouch! That's my fault! Thanks for reporting this!

stirlingstout commented 7 years ago

?Oh, and the title bar shows Snap4Arduino 1.0.6-beta.

John


From: Joan Guill?n i Pelegay notifications@github.com Sent: 27 September 2016 09:11 To: edutec/Snap4Arduino Cc: John Stout; Author Subject: Re: [edutec/Snap4Arduino] Premature leaving page dialog (#74)

Last version is 1.1.6 beta-rc3 released on 19/08/2016.

Current site is http://snap4arduino.org/

You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/edutec/Snap4Arduino/issues/74#issuecomment-249796365, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACEAB_z0OjxMP0UXoQ189tbXOpV4nhCPks5quM-wgaJpZM4KHCM7.

This email was Virus checked by KGV Sophos UTM. itservices@kgv.ac.uk

[King George V Logo] John Stout Management Information Systems Coordinator Tel: 01704 530601 E-Mail: JSS@kgv.ac.uk Web: www.kgv.ac.uk

Information contained in this e-mail is intended for the use of the addressee only and is confidential. Any dissemination, distribution, copying or use of this communication without prior permission of the addressee is strictly prohibited.

Every effort has been made to ensure that any attachment to this mail does not contain virus. King George V College has taken every reasonable precaution to minimise this risk, neither it nor the sender can accept liability for any damage which you sustain as a result of software viruses. You should carry out your own virus checks before opening any attachments.

[Think Before You Print]

stirlingstout commented 7 years ago

Bernat

Does that mean you can reproduce the problem? Or have found the solution? If so I won't send you the xml.

John

bromagosa commented 7 years ago

Nope, that just means I messed up the version reporting in the About dialog, but I still need the XML to test what's going on with your project :)

stirlingstout commented 7 years ago

Here's the project. If I edit the block called (table from file called (filename)) I can Apply it as it currently, but if I drag the map block that's on the scripting area into the [report []] and do Apply I get asked if I want to leave the page.

I've put my SnapFiles on Github at stirlingstout/SnapFiles. So far it works really nicely with an Arduino and Snap4Arduino, but it would be nice to have the tables feature from the current Snap! (and keyboard editing) to display data logging files.

PrematureLeavingPage.zip

bromagosa commented 7 years ago

Hmm... I see, it does work in my PC on the latest dev version. Can you try with the version I'm linking?

http://snap4arduino.org/downloads/test/linux32.zip http://snap4arduino.org/downloads/test/linux64.zip http://snap4arduino.org/downloads/test/osx64.zip http://snap4arduino.org/downloads/test/win32.zip http://snap4arduino.org/downloads/test/win64.zip

stirlingstout commented 7 years ago

I tried the Win64 version and that doesn't give the error, but when I import the project it tells me it was made with a different version of Snap!

bromagosa commented 7 years ago

Yep, that's a known issue I'll resolve before publishing the next release.

So it seems it's fixed in my dev branch, I'll close this issue when I release it :)

Thanks for your patience!

stirlingstout commented 7 years ago

Fantastic: table support and keyboard editing are in the beta version you pointed me to!

bromagosa commented 7 years ago

yep :)

I'm just waiting to fix an annoying bug to publish it, we'll hopefully fix it tomorrow with @jguille2...

bromagosa commented 7 years ago

About to release 1.2, so I'm closing this.