search

wlanslovenija / PiplMesh

A social info portal for wireless networks.
http://dev.wlan-si.net/wiki/PiplMesh
Other
40 stars 19 forks source link

Comments. Second try. #313

Closed mfrlin closed 11 years ago

mfrlin commented 11 years ago

There is problem with choosing the right textarea from which to get message for submiting a comment. Check TODOs. #164

mitar commented 11 years ago

This pull request cannot be automatically merged.

mfrlin commented 11 years ago

I've fixed most of the things or added TODOs. You said I shouldn't bind comment data on comments but if I don't do that I can't update the time on comments. I've looked how Posts and Notifications update their times and they bind data on their elements and then retrieve that data so they get an object and call updateDate on them.

mitar commented 11 years ago

Please update from main repository.

mitar commented 11 years ago

You said I shouldn't bind comment data on comments

I think I said that you should not do it twice. :-)

I am not sure where are you having problems. Please do the way you believe it is correct and necessary and I will check.

So tell me when you do everything you think it is still necessary.

mfrlin commented 11 years ago

Time updating is working now.

mitar commented 11 years ago

Please update from main repository. So code in main repository changed, so you have to merge with new changes. If you have not done this in the past successfully, I suggest that you make a backup of your local repository and try to do it and do not push anything to GitHub if you do not succeed.