Review Request 111654: add support for the discourse

Giorgos Tsiapaliokas terietor at gmail.com
Fri Jul 26 18:34:23 UTC 2013



> On July 23, 2013, 1:02 p.m., Aaron J. Seigo wrote:
> > why a clone of the discourse repo on github? was this to ensure that changes that happen in upstream discourse do not cause unexpected problems? or do you have actual changes in your github clone?
> > 
> > the installation procedure needs to be recorded in the README, but i suppose i can do that once this is in.
> > 
> > one this patch is in, there will need to modifications made to the asset info API in bodega-server to include a link to the discussion forum. that implies adding a new service block to the config.json for things like the url to the discourse server.

>why a clone of the discourse repo on github? 

The repository is in github because at the time that I created I wasn't sure
if we will need it and then I just continued to use it. If github isn't good
for us I can move it to projects.kde.org.

>was this to ensure that changes that happen in upstream discourse do not cause unexpected problems? 
>or do you have actual changes in your github clone?

yes to both of them.

 The changes to my github clone are related to our encryption. Discourse wasn't
using bcrypt and since we are syncing the password hashes there was no other way in order to make
it possible for the users to log in.


- Giorgos


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111654/#review36379
-----------------------------------------------------------


On July 23, 2013, 10:20 a.m., Giorgos Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111654/
> -----------------------------------------------------------
> 
> (Updated July 23, 2013, 10:20 a.m.)
> 
> 
> Review request for Bodega.
> 
> 
> Description
> -------
> 
> This patch keeps in sync the DBs of the bodega-server and discourse.
> 
> It syncs,
> 
> a. the user accounts*
> b. for each asset that has a partner with id > 1000 it create a new forum which has a topic and a post
> 
> * there is a default account for the discourse
> username: forumadmin
> password: password
> 
> 
> Diffs
> -----
> 
>   server/bodegaDbHelper 273e34f 
>   sql/discourse.plsql PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/111654/diff/
> 
> 
> Testing
> -------
> 
> How to test it,
> 
> * git clone https://github.com/terietor/discourse.git
> * cd discourse
> * git checkout bodega
> * https://github.com/terietor/discourse/blob/master/docs/DEVELOPER-ADVANCED.md
> * cd bodega-server/server
> * make sql && make test-sql (your bodega user must be a superuser in order to enable the dblink extention)
> * then go back to discourse and do `bundle exec rails s`
> * go to localhost:3000
> 
> 
> Thanks,
> 
> Giorgos Tsiapaliokas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/active/attachments/20130726/38c0d62d/attachment.html>


More information about the Active mailing list