-
-
Notifications
You must be signed in to change notification settings - Fork 81
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
save doesn't work on the Online Web App #40
Comments
If you have windows then better install Otto Blockly, web app is just to give a try to the UI and copy the code generated, it is not functional. |
I understand that "as is" this project isn't designed to be used from the web ... but I'm looking to adapt/modify/add-to this project to be used from a web environment ... I am expecting to encounter changes along the way with the handling of connectivity and uploading ... i'll also have to add this issue to the list ... since it is a "key" function to have ...just thought anyone familiar with the code might know why this wasn't working ... that's all ... Thank You |
That's cool yeah lets find a way to make it work as web app, have you seen
the arduino agent?
…On Friday, July 31, 2020, Scotty Franzyshen ***@***.***> wrote:
If you have windows then better install Otto Blockly, web app is just to
give a try to the UI and copy the code generated, it is not functional.
I understand that "as is" this project isn't designed to be used from the
web ... but I'm looking to adapt/modify/add-to this project to be used from
a web environment ... I am expecting to encounter changes along the way
with the handling of connectivity and uploading ... i'll also have to add
this issue to the list ... since it is a "key" function to have ...just
thought anyone familiar with the code might know why this wasn't working
... that's all ...
Thank You
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#40 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACAF2NDEJXD2W7BTQRO4GLTR6MZHTANCNFSM4PQ3FDEQ>
.
--
Best Regards
----------------------------------------------------------------------
Camilo Parra Palacio / Product Design Engineer
<https://www.behance.net/cparrapa>/ ottodiy.com
|
this problem is not related to this project's code ... it also happens in the the "fontainejp/blocklino" project code ... but interesting it works correctly in their web demo (different code from main project ) ... so I am comparing the differences to see if we can fix-it here ... |
indeed @sfranzyshen it could be because i started this fork when Blocklino was version 1.5 after that he changed a lot adding a web demo that we do not really need but the blocks creator is really good we can start but using that factory block creator feature. Let me know if you see other relevant difference and we update in Otto Blockly |
the code to handle the save button(s) functionality is in the electron included files only ... and are not pulled in when the web (index.html) is parsed alone. So, we have a choice ...
The Blocklino project has chosen to do it the #1 approach ... but i'm going to try to accomplish #2 for the save file functions ... but once we get into the communication, building, and uploading stuff ... it might be better to split the code into two projects ... ??? I don't know about this ... my motivations are to eliminate the serial communication and compiler stages completely ... only needing to communicate over wifi and running in a interpreter environment (no compiler step) sending script over the wifi network ... so, for now, i'm just going to concentrate on adding the save file functions when running in a browser ... to "this" code branch ... and ignore everything else (for now) ... |
I have seen many changes within the code ... not only between the OttoDIY Blockly and Blocklino code ... But also between Blocklino 1.8 (electron) and Blocklino 1.8 (web) code ... I've reached out to the developer for insight ... The only really big change that I see ... is the stuff for themes ... |
I think if you want to have all only through Wifi that should be a new separately software repo that we can call ¨¨Otto Blockly Wifi¨ because if you get rid of the serial communication and compiler what happens with all the people and teachers that are using it as it is? As you said you can try to get as far as possible as one repo, without damaging other functionalities and if something breaks but you fix the save problem then that means you should split them. I just invite you as part of the organization so feel free to create a new repo with your ideas and test them once we are there and functional we can find ways to combine them back again or maybe it won´t be necessary at all. I also got confused when making Otto Blockly (electron) on what files to use, i recently push all the folders and files I use to pack the .exe installer and still the web works to test the blocks but yes for the save function and the ones you suggested better to separate we can always have a link inside Otto Blockly electron installed to open the web version as alternative. If the only big change from Blocklino is the themes it doesn't matter i actually do not like better as simple as we have already. Have a look at the factory in Blocklino they have a function to save blocks and test them immediately. Thanks happy to have you as part of the developer team! |
I have sent a pull request to address this and other problems to using blockly from the web browser ... |
It would be nice to update the code to include the changes from the newer "block factory" from Blocklino 1.8 ... as you said it's very nice ... do you know specifically what branch and version of blocklino was used to start this project ... I can see that there were changes made to the code before the initial uploads made here ... it would help identify the changes step by step ... |
Great i think you solved the save function @sfranzyshen for the blocks mode, but for the code mode still doesn't work to save "download" the .ino file. The block factory now opens that is good and from there i can start to show you how to create the blocks. Now i don't know if the block disappear after deleted or close the program but this might help beginners to create new blocks faster, i am trying to see where was this updated. Let me know if you find it. |
NOTE: The Web version ... doesn't open a "save dialog" popup like in the electron version (or like when opening files) ... it will just save the file directly to the default download location without a prompt ... check your download folder to see if the files are there ... or from the browser check the "downloads" from the pull-down menu ... If the files are not there ... What browser version and operating system are you using? because from windows 10 64bit chrome I'm able to save the ".bloc" files and the code files too ... both ".ino" and ".py" files ... and I can do it from both icons ... the "save" button on the main code view screen ... and the one from the pop-out code viewer on the right side of the screen ... Could you test saving code files from the web version of the original blocklino project page ... and see if you are able to save code files from there? https://fontainejp.github.io/blocklino.html Thanks |
Hi @sfranzyshen |
@cparrapa The electron version allows directly adding blocks back to blockly ... but the web version only allows saving to ".bf" block factory files ... so the more I look at the changes between the electron and the web versions ... the more I see why the split in code bases ... this maybe what needs to happen here also ... we may need to split the code into two projects ... one for electron ... and one for web. |
@cparrapa |
Hi @sfranzyshen |
I'm going to go ahead and close this issue since it has been handled ... and open a separate new issue for updating the block factory code to the 1.8 version ... |
I am unable from the web app version to use the save button to save block xml files ... the open button and opening files work fine ... I'm using windows 10 and chrome Version 84.0.4147.89 (Official Build) (64-bit) ... no noticeable warnings or errors appear on the console ... even when the button is pressed ...
The text was updated successfully, but these errors were encountered: