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(client): Display player's first character ped... #596

Merged
merged 10 commits into from
Dec 18, 2024

Conversation

SSnowly
Copy link
Contributor

@SSnowly SSnowly commented Oct 17, 2024

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

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

client/character.lua Outdated Show resolved Hide resolved
client/character.lua Outdated Show resolved Hide resolved
@Manason
Copy link
Contributor

Manason commented Oct 18, 2024

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.

client/character.lua Outdated Show resolved Hide resolved
@SSnowly
Copy link
Contributor Author

SSnowly commented Nov 15, 2024

Sooooo, whats up?

@D4isDAVID
Copy link
Member

Waiting on you to act on the remaining code suggestions.

@solareon
Copy link
Contributor

solareon commented Dec 1, 2024

@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 mafewtm and D4isDAVID December 1, 2024 08:30
client/character.lua Outdated Show resolved Hide resolved
@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
2 checks passed
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