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

Wanted System Cleanup #41

Closed
BoGuu opened this issue Jul 28, 2016 · 6 comments · Fixed by #79
Closed

Wanted System Cleanup #41

BoGuu opened this issue Jul 28, 2016 · 6 comments · Fixed by #79
Labels
enhancement For pull requests and issues that suggest an improvement to an existing feature.

Comments

@BoGuu
Copy link
Member

BoGuu commented Jul 28, 2016

Need's a bloody good cleanup

@BoGuu BoGuu added Patch 2 and removed Backlog labels Jul 28, 2016
@dbenson24
Copy link
Contributor

I actually rewrote this also, made it into a Public Variable system with a refresh rate similar to how I did dynamic market. Made it much more responsive when alot of players were online and reduced database load

@BoGuu BoGuu mentioned this issue Aug 4, 2016
@nearbyplayer
Copy link
Contributor

nearbyplayer commented Aug 4, 2016

So I know you're planning to fix this up and since dbenson24 has some fixes to performance/responsiveness... wouldn't it be easy to tackle #299 - Custom Jail Times [suggestion]? Just an idea...

@BoGuu
Copy link
Member Author

BoGuu commented Aug 4, 2016

@nearbyplayer

It's already 'cleaned', reworks are future things. What do you mean easy to tackle? Shouldn't make a difference

@nearbyplayer
Copy link
Contributor

nearbyplayer commented Aug 4, 2016

@BoGuu
Whoops, I forgot that you had done it a couple of days ago, hence why you referenced it... just disregard my first comment.

@BoGuu
Copy link
Member Author

BoGuu commented Aug 4, 2016

@nearbyplayer

Nono, not working on it in that sense. Just cleaning up the messy code. It'll get done in due course (infact it should only take minimal changes). However, only bugs will be carried over from the Arma Life repo to here, i.e, no milestone features etc. This is a totally separate project.

@nearbyplayer
Copy link
Contributor

@BoGuu
Alright, I was aware this was a separate repo. However, I was not aware only bugs would be carried over, thanks for clarifying. Well, anyways I'll just let you guys get back to work, thanks again.

@tkcjesse tkcjesse mentioned this issue Aug 29, 2016
@BoGuu BoGuu closed this as completed in #79 Sep 12, 2016
@BoGuu BoGuu added Closed and removed enhancement labels Sep 12, 2016
@Jawshy Jawshy added the enhancement For pull requests and issues that suggest an improvement to an existing feature. label Sep 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement For pull requests and issues that suggest an improvement to an existing feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants