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

LinkGenerator: allow semicolon destination prefix #256

Open
wants to merge 25 commits into
base: master
Choose a base branch
from

Conversation

dakujem
Copy link

@dakujem dakujem commented Mar 24, 2020

Allows passing semicolon-prefixed destinations to LinkGenerator::link calls:

(new LinkGenerator(...))->link(':Foo:Bar:baz');
(new LinkGenerator(...))->link(':Bar:baz');

// and also this:
(new LinkGenerator(...))->link($presenter->getAction(true));

--
closes #255

@dg dg force-pushed the master branch 3 times, most recently from e23d6da to 22d5246 Compare March 25, 2020 10:45
@dakujem dakujem force-pushed the feat/trim-semicolon-prefixed-links-in-link-generator branch from 4d7947a to 82ad558 Compare March 25, 2020 17:09
@dakujem
Copy link
Author

dakujem commented Mar 25, 2020

@dg I'd be grateful if this was merged & released promptly. It would be awesome. I see praparations for next release. This should be fully compatible.

@dg dg force-pushed the master branch 11 times, most recently from 4aeacfd to 06a6513 Compare April 2, 2020 00:57
@dg dg force-pushed the master branch 3 times, most recently from 3b21865 to 548c1db Compare April 20, 2020 21:58
@dg dg force-pushed the master branch 3 times, most recently from 18efdae to 3f51018 Compare May 11, 2020 22:55
@dakujem
Copy link
Author

dakujem commented May 20, 2020

Should I rebase this? Any plan to fix the issue?

@dakujem dakujem force-pushed the feat/trim-semicolon-prefixed-links-in-link-generator branch from 82ad558 to b293b75 Compare May 20, 2020 19:17
@dakujem
Copy link
Author

dakujem commented May 20, 2020

You should update your test config - PHP 7.1 is no longer your target.

@dakujem dakujem force-pushed the feat/trim-semicolon-prefixed-links-in-link-generator branch from b293b75 to 6f03449 Compare May 28, 2020 12:52
@dg dg force-pushed the master branch 2 times, most recently from ef333e6 to 9d529b4 Compare June 9, 2020 12:55
@dg dg force-pushed the master branch 2 times, most recently from a1e8c9b to df7c4c9 Compare January 21, 2024 23:50
@dg dg force-pushed the master branch 2 times, most recently from 891b6dd to e3d05b3 Compare February 2, 2024 17:51
@dg dg force-pushed the master branch 3 times, most recently from 426e735 to c19ebdc Compare March 11, 2024 20:02
@dg dg force-pushed the master branch 5 times, most recently from 2b9da37 to 30d90f4 Compare April 7, 2024 02:51
@dg dg force-pushed the master branch 6 times, most recently from bf86204 to c91f90a Compare April 20, 2024 00:46
@dg dg force-pushed the master branch 3 times, most recently from 57bd587 to e908315 Compare May 2, 2024 10:37
@dg dg force-pushed the master branch 8 times, most recently from c5ecbda to ecb200c Compare May 13, 2024 09:25
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.

Unsupported semicolon prefix in LinkGenerator::link calls
3 participants