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.

Sign up for GitHub

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

Jump to bottom

Reworking Ability Use Limit System (From EHR) #1222

Draft
wants to merge 27 commits into
base: dev_2.2.0
Choose a base branch
from
Draft

Reworking Ability Use Limit System (From EHR) #1222

wants to merge 27 commits into from

Conversation

Tommy-XL
Copy link
Collaborator

No description provided.

@Tommy-XL Tommy-XL added Improve Code Port from EHR Some code/roles portsd from EHR in TOHE In Progress Pull request in progress labels Sep 23, 2024
@Tommy-XL
Copy link
Collaborator Author

Tommy-XL commented Sep 23, 2024 •
edited
Loading

There is no point in having every local ability for every role and doing a bunch of RPCs to sync with clients
It is better to combine them into one Dictionary which will always store information about this, and also have a common RPC for this

@0xDrMoe
Copy link
Member

0xDrMoe commented Sep 23, 2024

can we do this after 2.1.0?
I want canary out

@MargaretTheFool
Copy link
Contributor

oh hey I can finally code a role I was thinking of coding once this is finished

@Ultradragon005
Copy link
Contributor

image

@Pietrodjaowjao
Copy link
Contributor

Im Skibidier than you Drakos

@Pietrodjaowjao
Copy link
Contributor

me:
skibidi-toilet-titan-tv

@Ultradragon005
Copy link
Contributor

Im Skibidier than you Drakos

nooooooooooooooooooooooooooooooooooo 😭

@Ultradragon005
Copy link
Contributor

can we do this after 2.1.0? I want canary out

https://www.youtube.com/watch?v=tmL5ugYA9G0

@Tommy-XL Tommy-XL changed the base branch from DevBuild_2.1.0 to main October 27, 2024 08:51
@MargaretTheFool MargaretTheFool mentioned this pull request Nov 3, 2024
Merged
@NikoCat233 NikoCat233 changed the base branch from main to dev_2.2.0 November 22, 2024 18:08
@NikoCat233
Copy link
Collaborator

What does this do

@KingPanda360
Copy link
Contributor

What does this do

This is what it does: https://discord.com/channels/1094344790910455908/1289846310467403816

@NikoCat233
Copy link
Collaborator

What does this do

This is what it does: https://discord.com/channels/1094344790910455908/1289846310467403816

I dont have access to this channel

@KingPanda360
Copy link
Contributor

What does this do

This is what it does: https://discord.com/channels/1094344790910455908/1289846310467403816

I don't have access to this channel

Sent you the link in discord

@NikoCat233 NikoCat233 marked this pull request as ready for review December 6, 2024 10:45
@NikoCat233 NikoCat233 marked this pull request as draft December 6, 2024 10:45
@Tommy-XL
Copy link
Collaborator Author

Even though I'm no longer the main developer I still think I need to finish this, but looking at the amount of conflicts I want to cry💀

@NikoCat233
Copy link
Collaborator

bruh

@Tommy-XL
Copy link
Collaborator Author

Need do for:
Sniper, Berserker, Collector, CursedSoul, Infectious, Maverick, Pelican, Pirate, Pixie, Punching Bag

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Improve Code In Progress Pull request in progress Port from EHR Some code/roles portsd from EHR in TOHE
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Tommy-XL @0xDrMoe @MargaretTheFool @Ultradragon005 @Pietrodjaowjao @NikoCat233 @KingPanda360