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

Addition of "ScriptRelease" variable and it being displayed on the script console. #25

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

televisionia
Copy link

After looking through the code for a while, (it's insane! I praise your effort,) I decided to not bother about warning the user about the client/host having a different version and work on that some other time. I can already tell I don't have as much experience with sockets as you have. This pull request is meant to help users tell what version their script is to avoid the problems of connecting to a user with an outdated release, which has already happened (refer to #22 .)

I got ahold of how the script was working together, (it's a lot to take in,) but then I couldn't figure out what to change without breaking anything.

--Send Data to clients
function CreatePackett(RequestTemp, PackettTemp)
	local FillerStuff = "F"
	Packett = GameID .. Nickname .. PlayerID2 .. PlayerReceiveID .. RequestTemp .. PackettTemp .. CurrentX[PlayerID] .. CurrentY[PlayerID] .. Facing2[PlayerID] .. PlayerExtra1[PlayerID] .. PlayerExtra2[PlayerID] .. PlayerExtra3[PlayerID] .. PlayerExtra4[PlayerID] .. PlayerMapID .. PlayerMapIDPrev .. PlayerMapEntranceType .. StartX[PlayerID] .. StartY[PlayerID] .. FillerStuff .. "U"
end

Sadly I didn't have anyone to test my changes with remotely at the time, and I was trying to use my existing knowledge to include the release version of what script the user was using inside the packet being sent. Problem is, while I have Lua experience, I've mostly done this kind of stuff using python. This would be a handy feature to have in the next fix, just to make sure we don't get people confused. Because well, we already have enough people confused and lots of unclosed issues. Maybe you can do it before I can, edits are allowed for people with write access to the main branch.

If this is good, please review! (And merge, if you are the repo owner, yes, I see you)

@televisionia televisionia changed the title Pull request for adding a ScriptRelease variable with it being displayed in the console. Addition of "ScriptRelease" variable and it being displayed on the script console. Dec 10, 2022
@TheHunterManX
Copy link
Owner

I appreciate the input. I will not be merging, since Alpha 3.5 will be out this month.

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.

2 participants