> 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 > >
_______________________________________________ Active mailing list [email protected] https://mail.kde.org/mailman/listinfo/active
