As title, we need to firstly add a dependency on the views module, so that we can then properly test patches relating to the migration to lists with views.

CommentFileSizeAuthor
#1 storm-views-dependency-1844756-1.patch223 bytesAlex Andrascu

Comments

Alex Andrascu’s picture

Alex Andrascu commented
Status: Active » Needs review
StatusFileSize
new storm-views-dependency-1844756-1.patch223 bytes

Really interested to see this ported so fingers crossed expect more patches :)

Status: Needs review » Needs work

The last submitted patch, storm-views-dependency-1844756-1.patch, failed testing.

juliangb’s picture

juliangb commented
Status: Needs work » Fixed

Committed, thanks for getting involved (logged in this evening thinking i'd have to make this patch!)

(I knew this would fail the tests)

Look forward to seeing more patches! (Let me know if you'd like pointers for where to start...)

Alex Andrascu’s picture

Alex Andrascu commented

Yep...pointers would be good.
Any future plans for version control / repository integration ?

Thanks
Alex

juliangb’s picture

juliangb commented

Could you open a new issue about version control integration (if there isn't one), i'm not exactly sure what you mean so it'd be good to discuss.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.