search

happypeter / onestep

haoqicat.com 代码
http://haoqicat.com
171 stars 90 forks source link

有升级到Rails4的计划吗？啥时候升级？ #541

Closed diguage closed 10 years ago

diguage commented 10 years ago

有升级到Rails4的计划吗？啥时候升级？

happypeter commented 10 years ago

我们团队这边暂时没有时间，不过欢迎你发 PR

diguage commented 10 years ago

这个挺有挑战，不过可以尝试一下。哈哈

algo31031 commented 10 years ago

@happypeter 我试着搞了下, 在我fork的repo里加了个rails_4.1.1 branch 不过貌似github不支持直接PR到你这边的新分支里 不太清楚我该merge进我的master还是这边也加个新分支 rails_4.1.1, 哪个更好点

修改主要包括以下内容:

1. ruby及其他gem版本升级

ruby -> 2.1.2 rails -> 4.1.1 去掉Gemfile里各个gem的版本号

2. routes.rb

去掉了rails4弃用的match 部分put修改为patch 修改了少量重名routes(否则启动报错)

3. model修改

scope修改使符合rails4语法

4. jquery.Jcrop升级到0.9.12, 使之支持JQ 1.9
5. 顺便把#534 settings.yml.example 里缺的内容补上了
happypeter commented 10 years ago

@algo31031 太感谢了！

不太清楚我该merge进我的master还是这边也加个新分支 rails_4.1.1, 哪个更好点

可以放在你自己的 fork 上的 rails_4.1.1 分支上，然后给我发 PR，对比你自己的 rails_4.1.1 分支和 happypeter/onestep 的 master 分支就可以生成相应地 diff 了。

参考：不太清楚我该merge进我的master还是这边也加个新分支 rails_4.1.1, 哪个更好点

luckyyang commented 10 years ago

@algo31031 好厉害！

algo31031 commented 10 years ago

之前代码都很规整所以其实改的地方不太多 那些唬人的行数变化都是因为升级了jquery.Jcrop 库 - -!, 老版本的与jq1.9不兼容, 裁图那里用不了

luckyyang commented 10 years ago

我之前调试的时候就是因为当前的jquery版本与Jcrop的0.9.12不兼容，调试了半天才发现，很折磨，最后只能使用Jcrop v0.9.9

algo31031 commented 10 years ago

目前只算是初步可用的升级 rails4 的一个重大特性 strong parameters还没启用, 等有空再处理下

algo31031 commented 10 years ago

请先不要发布, font-awesome-rails这个gem遇到了问题, production环境下icon全没了 已经有人提了issue, 不过目前还没修正也没有人提出有效地解决办法, 先观望下吧

happypeter commented 10 years ago

:ok: