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.

Sign up for GitHub

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

Jump to bottom

Webhook Prints when missing #322

Merged
merged 5 commits into from
May 3, 2023
Merged

Webhook Prints when missing #322

merged 5 commits into from
May 3, 2023

Conversation

LeSiiN
Copy link
Contributor

@LeSiiN LeSiiN commented Apr 29, 2023

  • added prints so people know when missing a webhook

@LeSiiN
webhook error fix
893d43d 
- added prints so people know when missing a webhook
xFutte
xFutte reviewed Apr 29, 2023
View reviewed changes
client/cl_mugshot.lua Outdated Show resolved Hide resolved
xFutte
xFutte reviewed Apr 29, 2023
View reviewed changes
server/main.lua Outdated Show resolved Hide resolved
@xFutte
Copy link
Member

xFutte commented Apr 29, 2023

There really no reason to make that error so prominent in the console. Simply adding a single line with it is fine.

@LeSiiN
Copy link
Contributor Author

LeSiiN commented Apr 29, 2023

There really no reason to make that error so prominent in the console. Simply adding a single line with it is fine.

Are u sure, people cant even see the biggest errors in the console

@xFutte
Copy link
Member

xFutte commented Apr 29, 2023

Yes.

@LeSiiN
Copy link
Contributor Author

LeSiiN commented Apr 29, 2023

Yes.

Okay then

@xFutte
Copy link
Member

xFutte commented Apr 29, 2023

Feel free to add a color to the log

@LeSiiN
smaller prints
acb694c 
smaller prints
@LeSiiN
Copy link
Contributor Author

LeSiiN commented Apr 29, 2023

Feel free to add a color to the log

Wait in Color? How?

@xFutte
Copy link
Member

xFutte commented Apr 29, 2023

Feel free to add a color to the log

Wait in Color? How?

Ask ChatGPT

@LeSiiN
added color for prints
f0b8c1e 
added color for prints
@LeSiiN
added prints for onResourceStart
2ff88c8 
added prints for onResourceStart
@LeSiiN
added wait(3000) to be ad the bottom
1111941 
added wait(3000) to be ad the bottom
@MonkeyWhisper MonkeyWhisper changed the title webhook error fix Webhook Prints when missing May 3, 2023
@MonkeyWhisper MonkeyWhisper merged commit f19cca6 into Project-Sloth:main May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xFutte xFutte xFutte approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@LeSiiN @xFutte @MonkeyWhisper