Skip to content

Fix extra parameters for share tool, fixes #1494#1495

Merged
mattstauffer merged 1 commit intolaravel:masterfrom
skauk:cloudflared
Nov 25, 2024
Merged

Fix extra parameters for share tool, fixes #1494#1495
mattstauffer merged 1 commit intolaravel:masterfrom
skauk:cloudflared

Conversation

@skauk
Copy link
Copy Markdown
Contributor

@skauk skauk commented Sep 16, 2024

There was a typo in my previous PR that led to a problem passing extra arguments to share tools. Fixes #1494

@drbyte
Copy link
Copy Markdown
Contributor

drbyte commented Sep 16, 2024

Yes, that seems to be the right fix.
/cc @mattstauffer

@drbyte
Copy link
Copy Markdown
Contributor

drbyte commented Sep 16, 2024

@rcknr Can you edit your post above, and change See #1494 to Fixes #1494. That way Github will automatically close the related issue when this PR is merged (it uses the words "fixed" or "fix" or "fixes", or "closes", in the original PR post, to automatically trigger that).

@skauk skauk changed the title Fix extra parameters for share tool, #1494 Fix extra parameters for share tool, fixes #1494 Sep 16, 2024
@kiritokatklian
Copy link
Copy Markdown

Any updates on when we can expect this to be merged? 😄

@drbyte
Copy link
Copy Markdown
Contributor

drbyte commented Nov 6, 2024

//cc @mattstauffer

@mattstauffer mattstauffer merged commit 5a55c4a into laravel:master Nov 25, 2024
@mattstauffer
Copy link
Copy Markdown
Collaborator

Thanks for the PR @rcknr!

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.

Passing arguments to Ngrok broken in v.4.8.0

4 participants