Skip to content

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

@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