bay6 / union

rubyist community to help each other learn and improve
9 stars 8 forks source link

set Union ranking priviledge #165

Closed biorainy closed 11 years ago

biorainy commented 11 years ago

I can't test it in my local environment because I can't log in via github. When I logged in with admin account, the user seems to be nil and still can't test it.

Then I tried to deploy it to github, but failed in the beginning because it is complaining about bootstrap and jquery, after moving them to top level it worked but still can' t logged in because github complaining about different uri.

Anyway... please help me test the changes I made to 'app/controllers/records_controller.rb' file and just ignore the others. Just in case I messed up, I created another branch for this... You can consider pull this to another branch before making sure it works.

mouse-lin commented 11 years ago

why rm twitter-bootstrap-rails?

songjiayang commented 11 years ago

@biorainy 看看是否用cancan统一处理,比较方便. 比如类似于

def guest(user)
  cannot :action controller
end  
hlee commented 11 years ago

这个。。。我看是不是,我们快需要stage环境了呢?是不是在stage上跑跑再上线啊,正好可以给mouse做cap deploy练手用?

hlee commented 11 years ago

我手抖啊,不太敢merge,等胆大的上吧,嘿嘿

biorainy commented 11 years ago

我在本地运行,没法用github登陆。。有没有人知道原因?怎么解决?

On Mon, Jul 15, 2013 at 8:05 AM, Ken notifications@github.com wrote:

我手抖啊,不太敢merge,等胆大的上吧,嘿嘿

— Reply to this email directly or view it on GitHubhttps://github.com/bay6/union/pull/165#issuecomment-20965271 .

biorainy commented 11 years ago

@mouseshi 后来重新加上了,我自己本地 没法登陆,所以试着deploy到heroku上,结果heroku说bootstrap和jquery找不到,后来google下,说是要从assets里挪到top level。。。ignore那些吧。。都改回来了。。

On Mon, Jul 15, 2013 at 5:26 AM, mouseshi notifications@github.com wrote:

why rm twitter-bootstrap-rails?

— Reply to this email directly or view it on GitHubhttps://github.com/bay6/union/pull/165#issuecomment-20959206 .

hlee commented 11 years ago

本地运行起8080端口,没watch union? On Jul 15, 2013 11:21 AM, "Baichuan Li" notifications@github.com wrote:

我在本地运行,没法用github登陆。。有没有人知道原因?怎么解决?

On Mon, Jul 15, 2013 at 8:05 AM, Ken notifications@github.com wrote:

我手抖啊,不太敢merge,等胆大的上吧,嘿嘿

— Reply to this email directly or view it on GitHub< https://github.com/bay6/union/pull/165#issuecomment-20965271> .

— Reply to this email directly or view it on GitHubhttps://github.com/bay6/union/pull/165#issuecomment-20976785 .

songjiayang commented 11 years ago

LGMN 。。。。

hlee commented 11 years ago

你们这些家伙啊~,没有一个胆子大点儿的......

还得我老来。。。 可,可,我还是不敢,

@biorainy 那天晚上,或者周末有时间,来我公司,我们pair 一会儿,一块解决了吧,

biorainy commented 11 years ago

不用patch了。。。我测试了下不好使。。那回头我去找你

2013/7/16 Ken notifications@github.com

你们这些家伙啊~,没有一个胆子大点儿的......

还得我老来。。。 可,可,我还是不敢,

@biorainy https://github.com/biorainy 那天晚上,或者周末有时间,来我公司,我们pair 一会儿,一块解决了吧,

— Reply to this email directly or view it on GitHubhttps://github.com/bay6/union/pull/165#issuecomment-21037149 .