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

Revert "FIX Fluent now respects existing base URL prefixes in URL segment fields when no fluent domain exists" #539

Merged
merged 1 commit into from
Jul 30, 2019

Conversation

tractorcow
Copy link
Collaborator

Reverts #523

Resolves #538

Copy link
Collaborator

@ScopeyNZ ScopeyNZ left a comment

Choose a reason for hiding this comment

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

Straight revert - LGTM. @robbieaverill is off for a few days.

@ScopeyNZ ScopeyNZ merged commit 68b1b19 into 4.2 Jul 30, 2019
@ScopeyNZ ScopeyNZ deleted the revert-523-pulls/4.2/respect-existing-urlprefix branch July 30, 2019 21:56
@ScopeyNZ
Copy link
Collaborator

BTW @tractorcow - you'll have to delete the branches off of packagist - I don't have maintainer status on packagist for this repo.

@robbieaverill
Copy link
Contributor

Do we need to reopen silverstripe/silverstripe-subsites#416 now?

@ScopeyNZ
Copy link
Collaborator

Yep! (I did already 😉)

@tractorcow
Copy link
Collaborator Author

Oh sorry, yes I should be careful about those branches.

@ScopeyNZ I'll add you to packagist :P

@tractorcow
Copy link
Collaborator Author

Done https://packagist.org/packages/tractorcow/silverstripe-fluent

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.

3 participants