search

vti / mojox-session

Session management for Mojo
Other
15 stars 10 forks source link

Mojox::session breaking with Mojolicious 4.11 #11

Open kthakore opened 11 years ago

kthakore commented 11 years ago

Test output: http://aimed.cc/foo.txt

vti commented 11 years ago

I know! I am sorry! I hate it so much :D

kthakore commented 11 years ago

Is it critical? If you can point to me where the problem is I can try fixing it.

vti commented 11 years ago

You have to read through Changes in Mojolicious distribution. I haven't even looked at it. So maybe it's easy!

kthakore commented 11 years ago

So .... I have been trying to find this bug. Do you know how I can get the output as the error and not a dump of the HTML page mojo generates?

On Mon, Jun 3, 2013 at 11:02 AM, Viacheslav Tykhanovskyi < notifications@github.com> wrote:

You have to read through Changes in Mojolicious distribution. I haven't even looked at it. So maybe it's easy!

— Reply to this email directly or view it on GitHubhttps://github.com/vti/mojox-session/issues/11#issuecomment-18847198 .

vti commented 11 years ago

Done!

kthakore commented 11 years ago

It is fixed?

On Thu, Jun 6, 2013 at 11:49 AM, Viacheslav Tykhanovskyi < notifications@github.com> wrote:

Done!

— Reply to this email directly or view it on GitHubhttps://github.com/vti/mojox-session/issues/11#issuecomment-19054642 .

vti commented 11 years ago

Yes. The problem was in the tests. Their API changed.

vti commented 11 years ago

And it's already on CPAN!