Skip to content

Conversation

Meyndflay
Copy link

@Meyndflay Meyndflay commented Jun 16, 2025

Description

This PR adds a client function that makes an entity the direction of another, and a shared function that sanitizes a string.

Motivation

I found that I've been using such functions very often, and thought that i'd be nice to have as part of the framework's functions.

Implementation Details


Usage Example

-- Client
local ped = PlayerPedId()
local obj = CreateObject(hash, 0.0, 0.0, 0.0, false, true, true) 

ESX.FaceEntity(ped, obj)

-- Shared
local myString = "This is a string! \'\""
local sanitizedString = ESX.SanitizeString(myString)

--  Niche uses but useful for when you need to sanitize strings.

PR Checklist

  • My commit messages and PR title follow the Conventional Commits standard.
  • My changes have been tested locally and function as expected.
  • My PR does not introduce any breaking changes.
  • I have provided a clear explanation of what my PR does, including the reasoning behind the changes and any relevant context.

@github-project-automation github-project-automation bot moved this to 🆕 To be Discussed in ESX Roadmap Jun 16, 2025
@Mycroft-Studios
Copy link
Contributor

I like it , though am going to make as draft, since we are going to be doing an area specifically for functions like these very shortly :)

@github-project-automation github-project-automation bot moved this from 🆕 To be Discussed to In-Development in ESX Roadmap Jun 17, 2025
@Meyndflay Meyndflay requested a review from Kenshiin13 June 18, 2025 09:55
@github-project-automation github-project-automation bot moved this from In-Development to Ready for Next Update in ESX Roadmap Jun 20, 2025
@Kenshiin13 Kenshiin13 marked this pull request as draft July 6, 2025 08:38
@Kenshiin13
Copy link
Contributor

@Meyndflay There soon will be a place for these changes 👍🏽 Thanks!

@Kr3mu Kr3mu requested a review from Kenshiin13 September 2, 2025 09:46
@Kr3mu
Copy link
Member

Kr3mu commented Sep 2, 2025

@Meyndflay There soon will be a place for these changes 👍🏽 Thanks!

We need to finally implement it or reject/change

@Kenshiin13
Copy link
Contributor

We need to finally implement it or reject/change

We are waiting for the lib.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Ready for Next Update
Development

Successfully merging this pull request may close these issues.

4 participants