Review Request 111654: add support for the discourse

[Giorgos Tsiapaliokas]

> On July 23, 2013, 1:02 p.m., Aaron J. Seigo wrote:
> > sql/discourse.plsql, lines 152-157
> > <http://git.reviewboard.kde.org/r/111654/diff/1/?file=173010#file173010line152>
> >
> >     the names are not guaranteed to be unique between assets.
> >     
> >     if two people have an asset named "Awesome" this will cause problems.
> >     
> >     i would recommend storing the category and topic id's from the discourse db in bodega db or alternatively (and this would probably be nicer) add a reliable id to the topics/categories in the the discourse db. (e.g. based on the asset's ID, which never changes)
> >     
> >     this could be done non-intrusively by adding another table to the discourse db where these "bodega" ids are stored, perhaps?

a new table named bodegaAssets has been added to the discourse db.


- 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/20130727/7502087d/attachment.html>