Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

Cherry pick fixes for #51 and #58 (closes #64) #65

Merged
merged 2 commits into from
Sep 11, 2014
Merged

Cherry pick fixes for #51 and #58 (closes #64) #65

merged 2 commits into from
Sep 11, 2014

Conversation

vbalazs
Copy link

@vbalazs vbalazs commented Sep 10, 2014

Backport fixes for login issues caused by custom route mount point.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling dfb32e0 on digitalnatives:v2_maintenance into 21ef582 on rbCAS:2.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling dfb32e0 on digitalnatives:v2_maintenance into 21ef582 on rbCAS:2.0.

@vbalazs
Copy link
Author

vbalazs commented Sep 10, 2014

@pencil What do you think? Should I look into backporting other bugfixes or is it enough for a 2.0.7 release? :)

@pencil
Copy link
Member

pencil commented Sep 10, 2014

Thank you! Think it should be enough for a bugfix release. Did you test it though?

@vbalazs
Copy link
Author

vbalazs commented Sep 10, 2014

Yes, these two commits solved my issues and the generic functionality still works. I haven't done full manual testing though.

Later tonight I'll test it with root mount point as well and get back to you. You can hold the merge until then.

@vbalazs
Copy link
Author

vbalazs commented Sep 10, 2014

@pencil I tested it with root mount point, it seems OK to me.

pencil added a commit that referenced this pull request Sep 11, 2014
@pencil pencil merged commit 74509fe into rbCAS:2.0 Sep 11, 2014
@pencil
Copy link
Member

pencil commented Sep 11, 2014

Thanks. Just released 2.0.7. 🍻

@vbalazs vbalazs deleted the v2_maintenance branch October 22, 2014 10:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants