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

Feat: ESX Implementation #92

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Conversation

FBFezz
Copy link

@FBFezz FBFezz commented Feb 28, 2025

I have added esx (basically just converted the qbcore functions to esx)
I have tested this but i would double check just in case.

Copy link
Collaborator

@Mathu-lmn Mathu-lmn left a comment

Choose a reason for hiding this comment

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

Looks mostly good to me, tho I think the frameworks file in the client side need a refactor but for now it's ok.
Just change my 2 comments and we can merge

@@ -47,7 +47,10 @@ if Config.ExpressionsEnabled then

if Config.PersistentExpression then
AddEventHandler('playerSpawned', LoadPersistentExpression)
RegisterNetEvent('QBCore:Client:OnPlayerLoaded', LoadPersistentExpression)
RegisterNetEvent('esx:playerLoaded', LoadPersistentExpression)
if Config.Framework == 'qb-core' then
Copy link
Collaborator

Choose a reason for hiding this comment

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

Those checks are not useful since we are currently registering those events listener, if the server is qb-core, then the esx event will simply never be triggered

client/Walk.lua Outdated
RegisterNetEvent('QBCore:Client:OnPlayerLoaded', handleWalkstyle)
RegisterNetEvent('esx:playerLoaded', handleWalkstyle)

if Config.Framework == 'qb-core' then
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same as above

@FBFezz
Copy link
Author

FBFezz commented Feb 28, 2025

Both should be reverted back now

@FBFezz FBFezz requested a review from Mathu-lmn February 28, 2025 12:04
@alberttheprince alberttheprince removed the request for review from enzo2991 March 1, 2025 07:45
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