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

Update to use the native qb-core Round function #57

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

mbiddle
Copy link
Contributor

@mbiddle mbiddle commented Nov 15, 2024

Describe Pull request
Updating to use the native qb-core Round function. I didn't realize this function existed in my previous PR that introduced it.

If your PR is to fix an issue mention that issue here

Questions (please complete the following information):

  • Have you personally loaded this code into an updated qbcore project and checked all it's functionality?
  • Does your code fit the style guidelines?
  • Does your PR fit the contribution guidelines?

…his function existed in my previous PR that introduced it.
Copy link
Contributor

@Cocodrulo Cocodrulo left a comment

Choose a reason for hiding this comment

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

That's a better approach, good job!

Copy link

@Qwerty1Verified Qwerty1Verified left a comment

Choose a reason for hiding this comment

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

I did question it, but forgot we even had it.

@GhzGarage GhzGarage merged commit 903a9ee into qbcore-framework:main Nov 19, 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.

4 participants