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

AFK #230

Open
wants to merge 13 commits into
base: repo/1.5.3
Choose a base branch
from
Open

AFK #230

wants to merge 13 commits into from

Conversation

daniiv19
Copy link

Anti AFK. An extension that tracks how long you have been AFK.

@WiredSpast
Copy link
Collaborator

Your files are in the wrong folder, they should be in store/extensions/AFK/

@WiredSpast
Copy link
Collaborator

https://github.com/daniiv19/anti-afk/blob/main/main.go#L93-L94

You're sending an outgoing shout, best to use an incoming one, otherwise it's very obvious that the user is scripting.

Remove all print statements before compiling it to an exe, G-Earth runs extensions without an attached console, so print statements cause the extension to throw errors.

Updated folder location and it's content.
@daniiv19
Copy link
Author

https://github.com/daniiv19/anti-afk/blob/main/main.go#L93-L94

You're sending an outgoing shout, best to use an incoming one, otherwise it's very obvious that the user is scripting.

Remove all print statements before compiling it to an exe, G-Earth runs extensions without an attached console, so print statements cause the extension to throw errors.

I changed the main branch, I'll update the .zip folder in exctensionStore

daniiv19 added 2 commits July 18, 2024 15:06
updated it contents to the latest version.
Updated zip content:
Removed README.txt
Added text to README.md
@WiredSpast
Copy link
Collaborator

I changed the main branch, I'll update the .zip folder in exctensionStore

https://github.com/daniiv19/anti-afk/blob/main/main.go#L87
Have you tested if it works now, because the incoming normally has a number before the message

@daniiv19
Copy link
Author

I changed the main branch, I'll update the .zip folder in exctensionStore

https://github.com/daniiv19/anti-afk/blob/main/main.go#L87 Have you tested if it works now, because the incoming normally has a number before the message

Should work now after few tests, this version is the most stable.

"name": "danii / v19",
"hotel": ".com",
"username": "v19",
"discord": "danii98",
Copy link
Collaborator

Choose a reason for hiding this comment

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

This comma is breaking the parsing, remove it

Suggested change
"discord": "danii98",
"discord": "danii98"

@WiredSpast
Copy link
Collaborator

Huh apparently extensions need to have a name of 4 or more characters, didn't know that.
image
(Reply with a comment when you've fixed that, I don't get notifications from just pushes)

@daniiv19
Copy link
Author

Updated the name, zip content and main.go file.

@WiredSpast
Copy link
Collaborator

Remove the old folder and make sure the name in the extension.json matches the folder name

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