search

colinmollenhour / Cm_RedisSession

Redis-based session handler for Magento with optimistic locking
208 stars 121 forks source link

GitHub 127 installs into app code local cm redis session code #128

Closed kirkmadera closed 6 years ago

kirkmadera commented 7 years ago

Most modules are now structured like this where the path matches the target path. I think the only real requirement is that the final filename matches the existing one for it to now make a new subdirectory, however, this is the most common approach for M1 modules.

kirkmadera commented 7 years ago

Resolves #127

colinmollenhour commented 7 years ago

This seems like a reasonable change. I'll try to test it out soon.

Thanks!

kirkmadera commented 7 years ago

I just thought about this also. This change could be a good opportunity to switch from app/code/local to app/code/community also.

seansan commented 6 years ago

@kirkmadera - If this is merged + moved to community then adding some kind of simple PHP script to remove the old files wouold be nice. Reason 1: it makes it simple. But IK ave learnt reason 2: people see the file and are made more aware of the move in codepool.

colinmollenhour commented 6 years ago

Merged and tagged as 2.2.0