Skip to content
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

Lua patch: revamping installing Lua Workout #3298

Open
wants to merge 72 commits into
base: master
Choose a base branch
from

Conversation

junoocha
Copy link
Collaborator

This workout was heavily revamped due to the many confusions users had with actually downloading Lua and using it on their computer.

As such, I considered an alternative for users to easily code on Lua. I realized that most people are here to be able to code on Roblox as that is the most popular platform for Lua coding and as such, downloading Lua is unnecessary as the Lua compiler is built into Roblox Studio. Thus, simply having an online IDE would be sufficient for following along in this course.

For users that do want to install Lua, I did make some clarifications with installing Lua and suggested that for any specific issues, watching tutorials would help, as its pretty difficult for new users to make the changes on their computer and coding environment. I did make a notice on insight 2 that insights 2-4 can be skipped if the user just plans to code on Roblox.

Additionally, I found an android app that also served as a Lua ide, allowing users to code Lua on their phones. A lot of people were asking for a mobile IDE to play around with Lua on there, and as I know that the current Enki app doesn't support Lua code, this app may serve as an alternative to coding on the go. I unfortunately don't have access to any apple products, but I do know of some Lua Ides on the apple store. I haven't put them in yet as I have not used them. I can put them in with further review.

On a side note, with this changes, a note may have to be added for input/output operations, as it does involve writing and exporting files and cannot be done on an online IDE.

@enki-hq
Copy link
Collaborator

enki-hq commented Mar 22, 2024

Insight previews:


For the interpreter, you can easily find it on the download page of the [official Lua website](https://www.lua.org/download.html).
For this course, coding on **an online IDE[1] is perfectly fine**! You can view online demo programs here: [live demo](https://www.lua.org/demo.html).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
For this course, coding on **an online IDE[1] is perfectly fine**! You can view online demo programs here: [live demo](https://www.lua.org/demo.html).
For this course, coding on **an online IDE[1] is perfectly fine**! You can view online demo programs [here](https://www.lua.org/demo.html).

> Learning Lua will help you build Roblox games and more, which is why we suggest it as your starting point.
> However, if you'd like, you can skip Lua and jump into our [Roblox Studio course](https://app.enki.com/course/roblox-studio).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
> However, if you'd like, you can skip Lua and jump into our [Roblox Studio course](https://app.enki.com/course/roblox-studio).
> If you'd like, you can skip Lua and jump into our [Roblox Studio course](https://app.enki.com/course/roblox-studio).

@enki-hq
Copy link
Collaborator

enki-hq commented May 8, 2024

Insight

Insight Name Error Message
python-environment
  • Install Python pointing to https://wiki.python.org/moin/BeginnersGuide/Download is broken because HTTP_503

If you notice anything wrong with these rules contact content-team@enki.com.

@junoocha
Copy link
Collaborator Author

junoocha commented May 8, 2024

New Changes:

  • Added the commits from Nem
  • For insight 2, I moved the note regarding the optionality of the next 3 insights to the top, as it should be the first thing a user should read before deciding
  • Removed the mention of LuaDroid to keep the focus on Enki

@junoocha
Copy link
Collaborator Author

junoocha commented May 8, 2024

New Changes:

  • Added the commits from Nem
  • For insight 2, I moved the note regarding the optionality of the next 3 insights to the top, as it should be the first thing a user should read before deciding
  • Removed the mention of LuaDroid to keep the focus on Enki

I might also consider removing most of the content in this workout, as it's extremely likely that users will be coding on Roblox to begin with, hence there's no need for this portion. IF the change does go through, I'll likely keep the online demo of the Lua Code, and then directing to the Roblox Studio insight where users can start scripting/coding.

@junoocha
Copy link
Collaborator Author

Reverted the changes regarding deletion of files to properly archive them in a different PR.

The plan is to completely remove most of the installation workout and move the first insight to the introduction workout. This first insight was edited to explain that the online IDE is enough for the course, and also directed them to the Roblox Studio intro to scripting workout if they'd like to code through Studio (as it comes with Lua installed)

Additionally, as the course has now switched to the use of an online IDE, there is no use for IO operations. This is furthered by the fact that these operations are deprecated in Roblox Studio. Thus, the two workouts that explore IO operations will also be archived.

@enki-hq
Copy link
Collaborator

enki-hq commented May 21, 2024

Insight

Insight Name Error Message
installing-lua-i Insight slug installing-lua-i is not unique in the database.

If you notice anything wrong with these rules contact content-team@enki.com.

junoocha and others added 2 commits May 23, 2024 13:21
Co-authored-by: Stefan Stojanovic <Stefan-Stojanovic@users.noreply.github.com>
Co-authored-by: Stefan Stojanovic <Stefan-Stojanovic@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants