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

Fix account issues + qb-management export support #10

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Millusion
Copy link

no idea why he is not merging this in...so i'm doing it

@SamShanks1
Copy link
Contributor

is this tested and working @RoccoMyTacco @Millusion

@RoccoMyTacco
Copy link
Contributor

RoccoMyTacco commented Oct 24, 2022

is this tested and working @RoccoMyTacco @Millusion

uh last time I tested it, it worked fine but I didnt test too much

Was also waiting for this to get done project-error/pefcl#98

@Millusion
Copy link
Author

for now on my local dev build i have no issues with this

Copy link

@Mkeefeus Mkeefeus left a comment

Choose a reason for hiding this comment

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

reason wasn't showing in the UI when I tried it, looks like the arg name was wrong

local data = {
identifier = account,
amount = amount,
description = reason

Choose a reason for hiding this comment

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

Suggested change
description = reason
message = reason

server.lua Outdated Show resolved Hide resolved
server.lua Outdated Show resolved Hide resolved
@lhuanluz
Copy link

you are a god, u just fixed everything for me!

RoccoMyTacco and others added 2 commits July 14, 2023 00:55
Co-authored-by: Malcolm Keefe <27903055+Mkeefeus@users.noreply.github.com>
Co-authored-by: Malcolm Keefe <27903055+Mkeefeus@users.noreply.github.com>
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.

5 participants