Skip to content

Update send.lua - #2339

Merged
z16 merged 7 commits into
devfrom
lili-ffxi-patch-1
Apr 3, 2024
Merged

Update send.lua#2339
z16 merged 7 commits into
devfrom
lili-ffxi-patch-1

Conversation

@lili-ffxi

Copy link
Copy Markdown
Contributor

Adjust string processing to reinstate double quotes removed by the command processor.

Adjust string processing to reinstate double quotes removed by the command processor.
@lili-ffxi

Copy link
Copy Markdown
ContributorAuthor

Bug found. Will reopen once fixed.

@lili-ffxilili-ffxi reopened this Mar 5, 2024
@lili-ffxi

Copy link
Copy Markdown
ContributorAuthor

Fixed, I had swapped two arguments because of course I did.

Spaces after commas.
Fixed an oversight around the `<tid>` functionality.
@z16z16 mentioned this pull request Apr 3, 2024
@zmsl

zmsl commented Apr 3, 2024

Copy link
Copy Markdown
Contributor

I am attempting to load this version into my XI client and am getting an error when running a simple send

image

Command: "send Altname test"

@lili-ffxi

Copy link
Copy Markdown
ContributorAuthor

I am attempting to load this version into my XI client and am getting an error when running a simple send

image

Command: "send Altname test"

This error means that you downloaded the addon manually and tried to run it on windower live. You need windower dev for this to work.

Added toggle command for the debug functionality.
Rename dbg to debug
@z16
z16 merged commit b65625c into devApr 3, 2024
@z16
z16 deleted the lili-ffxi-patch-1 branch April 3, 2024 21:29
z16 pushed a commit that referenced this pull request Nov 4, 2024
* Update send.lua
Adjust string processing to reinstate double quotes removed by the command processor.
* Update send.lua
* Update send.lua
Spaces after commas.
* Update send.lua
Fixed an oversight around the `<tid>` functionality.
* Update send.lua
Added toggle command for the debug functionality.
* Update send.lua
* Update send.lua
Rename dbg to debug
Sign up for freeto 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

@lili-ffxi@zmsl@z16