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: improvements of invoice system #398

Merged
merged 6 commits into from
Nov 13, 2024
Merged

Conversation

Z3rio
Copy link
Contributor

@Z3rio Z3rio commented Jun 24, 2024

Description

Changes:

  • Secured unsecured invoice events
  • Added reason column to invoices to specify the reasoning behind the invoice
  • Added candecline column to invoices to specify whether the invoice can be declined or not
  • Added an event that gets triggered upon an invoice being paid, or declined
  • Fixed syncing of invoices, they are now fetched upon opening the bank app, instead of fetching them one singular time when opening the phone for the first time.

Checklist

  • I have personally loaded this code into an updated qbcore project and checked all of its functionality.
  • My code fits the style guidelines.
  • My PR fits the contribution guidelines.

Copy link

This PR has had 60 days of inactivity & will close within 7 days

@github-actions github-actions bot added the Stale label Aug 24, 2024
@Z3rio
Copy link
Contributor Author

Z3rio commented Aug 25, 2024

This PR has had 60 days of inactivity & will close within 7 days

Yeah, no. QBCore is the ones that are "inactive", not me lmfao. Thanks

Genuinely no point in this bot, doing way more damage than good.

@github-actions github-actions bot removed the Stale label Aug 25, 2024
@Z3rio
Copy link
Contributor Author

Z3rio commented Nov 12, 2024

Would be great to have this looked at, would love to be able to clean-up some of my old qb repos.
@GhzGarage

server/main.lua Outdated Show resolved Hide resolved
@GhzGarage GhzGarage merged commit 007e5ee into qbcore-framework:main Nov 13, 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.

3 participants