Skip to content

feat(client): Display player's first character ped...#596

Merged
Manason merged 10 commits into
Qbox-project:mainfrom
SSnowly:main
Dec 18, 2024
Merged

feat(client): Display player's first character ped...#596
Manason merged 10 commits into
Qbox-project:mainfrom
SSnowly:main

Conversation

@SSnowly

@SSnowly SSnowly commented Oct 17, 2024

Copy link
Copy Markdown
Contributor

Description

Display's the player's first character -if it exists- instead of a random ped. If there is no characters, it displays a random Ped as the default does.
The video: in discord

Checklist

  • [ x ] I have personally loaded this code into an updated Qbox project and checked all of its functionality.
  • [ x ] My pull request fits the contribution guidelines & code conventions.

@dasbootdev

Copy link
Copy Markdown

Pulled into my latest qbox unstable dev server and it's working as intended, looks great

Comment thread client/character.lua
Comment thread client/character.lua Outdated
@Manason

Manason commented Oct 18, 2024

Copy link
Copy Markdown
Contributor

I think it might be best if we just don't display any character if none exist. Although that could be out of scope for this PR.

Comment thread client/character.lua Outdated
@SSnowly

SSnowly commented Nov 15, 2024

Copy link
Copy Markdown
Contributor Author

Sooooo, whats up?

@D4isDAVID

Copy link
Copy Markdown
Member

Waiting on you to act on the remaining code suggestions.

@solareon

solareon commented Dec 1, 2024

Copy link
Copy Markdown
Contributor

@SSnowly fixed the changes for you. can you test and report back please

Manason
Manason previously approved these changes Dec 1, 2024
@solareon solareon requested review from D4isDAVID and mafewtm December 1, 2024 08:30
Comment thread client/character.lua Outdated
@Manason Manason requested a review from D4isDAVID December 7, 2024 17:08
@D4isDAVID D4isDAVID requested a review from Manason December 17, 2024 23:16
@Manason Manason merged commit f95bc2d into Qbox-project:main Dec 18, 2024
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.

6 participants