Skip to content

Comments

Added Inheritances and Exports | Fixed Embed not Sending when you change blips while on-duty.#6

Open
NickReagan wants to merge 6 commits intoJaredScar:masterfrom
NickReagan:master
Open

Added Inheritances and Exports | Fixed Embed not Sending when you change blips while on-duty.#6
NickReagan wants to merge 6 commits intoJaredScar:masterfrom
NickReagan:master

Conversation

@NickReagan
Copy link

@NickReagan NickReagan commented Feb 7, 2023

Been need Inheritances for awhile so I spent a few hours and did it :P
Also made a PR for your docs @JaredScar

@NickReagan
Copy link
Author

There is a lot of nested for loops... I am trying to think of ways to optimize it but it works lol. At least the code doesn't run 24/7 and just on player join or resource restart.

Also on resource restart it goes through the online players and registers them and waits half a second before doing the next player. Hopefully that is enough for it not to get rate limited.

@NickReagan
Copy link
Author

Everything seems to be working fine.

@NickReagan NickReagan marked this pull request as draft February 11, 2023 01:18
@NickReagan NickReagan marked this pull request as ready for review March 3, 2023 16:41
@NickReagan
Copy link
Author

Done extensive testing on my live server. Good to go.

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