Skip to content

feat(progress): ptfx for props#89

Open
Andyyy7666 wants to merge 1 commit intoCommunityOx:mainfrom
Andyyy7666:patch-1
Open

feat(progress): ptfx for props#89
Andyyy7666 wants to merge 1 commit intoCommunityOx:mainfrom
Andyyy7666:patch-1

Conversation

@Andyyy7666
Copy link

Added support for particle effects when creating props

Added support for particle effects when creating props

Signed-off-by: Andy <86536434+Andyyy7666@users.noreply.github.com>
@FjamZoo
Copy link

FjamZoo commented Mar 3, 2026

Last time you did a PR to progressbar it was so horrible that Rockstar had to make an announcement when it was patched. Do we have the balls to risk this disaster once again...

@Somis12
Copy link

Somis12 commented Mar 3, 2026

is that client side only particle?

@Andyyy7666
Copy link
Author

Last time you did a PR to progressbar it was so horrible that Rockstar had to make an announcement when it was patched. Do we have the balls to risk this disaster once again...

Hahahah shush, that was lindens fault. This is just adding ptfx.

@Andyyy7666
Copy link
Author

is that client side only particle?

It's synced with a state bag.

@@ -229,16 +253,28 @@

local function deleteProgressProps(serverId)
Copy link
Member

Choose a reason for hiding this comment

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

Being that this function no longer relates to just the progress props either rename it or split the removal of props and ptfx within separate functions. (Personal suggestion)

---@field prop? ProgressPropProps | ProgressPropProps[]
---@field disable? { move?: boolean, sprint?: boolean, car?: boolean, combat?: boolean, mouse?: boolean }

local function createProp(ped, prop)
Copy link
Member

Choose a reason for hiding this comment

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

Same basis as the comment for line 254

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.

4 participants