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

[Feature Request] - ADD STATE BAG - isLoggedIn #1348

Closed
Mesrine67 opened this issue Apr 20, 2024 · 6 comments
Closed

[Feature Request] - ADD STATE BAG - isLoggedIn #1348

Mesrine67 opened this issue Apr 20, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@Mesrine67
Copy link
Contributor

Add the isLoggedIn statebag for better management, especially for developers creating content compatible with multiple frameworks.

@Mesrine67 Mesrine67 added the enhancement New feature or request label Apr 20, 2024
@github-project-automation github-project-automation bot moved this to 🆕 New in ESX Roadmap Apr 20, 2024
@Thekuca
Copy link
Contributor

Thekuca commented Apr 20, 2024

ESX.PlayerLoaded already exists

@Mesrine67
Copy link
Contributor Author

yes but I ask for a statebag

@Gellipapa
Copy link
Contributor

Gellipapa commented Apr 21, 2024

yes but I ask for a statebag

Hi! It's a global client side variable, it works exactly like the statebag, you create a custom method to check the framework types and set the ESX.PlayerLoaded variable in your statebage in playerLoaded so you can use it for multiple frameworks.

I don't see the benefit of putting a global variable that is already client-side accessible in statebag.

Where are you stuck or why do you think a statebag is needed?

@Mesrine67
Copy link
Contributor Author

these so as not to have to go through the ESX variable and go through the statebag

@Gellipapa Gellipapa moved this from 🆕 New to 👀 In review in ESX Roadmap May 3, 2024
@FUst1ke
Copy link

FUst1ke commented May 19, 2024

You can still set your own state via two event handlers, one is onPlayerLoaded and the other is the playerDropped. Literally no need to add a state next to a client side function, you can register a state yourself too.

@Arctos2win
Copy link
Contributor

I really don't see any benefit of implementing this in esx legacy.

@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in ESX Roadmap May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Archived in project
Development

No branches or pull requests

6 participants