Skip to content
This repository has been archived by the owner on Feb 8, 2024. It is now read-only.

QB-Phone | Renewed | Imgur API integration #99

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ogidevs
Copy link

@ogidevs ogidevs commented Feb 9, 2023

As we all know discord cdn started blocking fivem's request for webhooks, so here is another solution.

FOR THIS TO WORK YOU WILL NEED MY FORK OF SCREENSHOT-BASIC:
citizenfx/screenshot-basic#41

Do the necessary changes for screenshot-basic and you are all set.

As we all know discord cdn started blocking fivem's request for webhooks, so here is another solution.

FOR THIS TO WORK YOU WILL NEED MY FORK OF SCREENSHOT-BASIC:
citizenfx/screenshot-basic#41
Do the necessary changes for screenshot-basic and you are all set.
@PEROXCYDE
Copy link

What changes you do to solve sql error?
SCRIPT ERROR in promise (unhandled rejection): Error: qb-phone was unable to execute a query!
Column 'image' cannot be null
INSERT INTO phone_gallery (citizenid, image) VALUES (?, ?) ["HJR73130",null,null]

@ogidevs
Copy link
Author

ogidevs commented Feb 9, 2023

By the error, this usually means that one did not follow the description, you have to set your imgur application id in screenshot-basic resource before running it for the frist time on the server like this:
Authorization: "Client-ID YOURCLIENTID"

@PEROXCYDE
Copy link

Screenshot_3

@ogidevs
Copy link
Author

ogidevs commented Feb 9, 2023

Again....
Authorization: "Client-ID YOURCLIENTID"

@PEROXCYDE
Copy link

thx buddy i'm pretty dumb, just cleared cache and it worked

@ogidevs
Copy link
Author

ogidevs commented Feb 9, 2023

No worries, glad to help!

@paruthepro
Copy link
Contributor

Works very well, can be merged :)

@Brosky01
Copy link

WHERE CAN I FIND THIS PLEAS Authorization: "Client-ID YOURCLIENTID" THANKS FOR THE PHOOOOONE <333

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants