13 May, 2016
13 commits
-
admin_notifications: ActiveRecord::Base to ApplicationRecord See merge request !909
-
Remove cube-responsive theme as it doesn't work See merge request !904
-
Remove ProfileMembersHeadlinesBlock#content This method was in violation of MVC principles and its logic is now handled by the BoxesHelper. Minor adaptations to the view and a rename were required in order to meet BoxesHelper expectations. See merge request !908
-
Remove TourBlock#content This method was in violation of MVC principles and its logic is now handled by the BoxesHelper. See merge request !907
-
Remove InterestsBlock#content method This removes html generation from the class improving MVC responsability separation. It required the creation of a new method, interests, and view move/rename. See merge request !916
-
Refactor breadcrumbs plugin The `ContentBreadcrumbsBlock` class HTML generation have been refactored (thanks @joenio) in terms of view and helper, improving MVC compliance but still missed the Controller coupling with the view and helper through the dependency for `params` variable. I'm still not 100% sure if `params` is accessible from views. Some more feedback and review on it would be welcome. Finally the unit tests have been fixed accordingly to the refactor. Travis build: https://travis-ci.org/rafamanzo/noosfero/builds/128041637 See merge request !898
12 May, 2016
7 commits
-
Add steps to upload image using people's endpoint With this merge, the frontend will be able to send a image(converted to base64) to the api,and save it in database. It's possible to extend this feature to other models that have image, as articles. See merge request !876
-
Api endpoint for blocks See merge request !913
-
This removes html generation from the class improving MVC responsability separation. It required the creation of a new method, interests, and view move/rename.
11 May, 2016
3 commits
10 May, 2016
17 commits
-
expose statistics block atrributes in api expose statistics block atrributes in api See merge request !902
-
comment_paragraph: add discussion content type See merge request !903
-
CommentParagraph: add data-macro-paragraph_uuid in sanitized allowed attributes After save TinyMceArticle article a worng parse was removing data-macro-paragraph_uuid from body data See merge request !912
-
api: expose accept_comments from article See merge request !910
-
This method was in violation of MVC principles and its logic is now handled by the BoxesHelper. Minor adaptations to the view and a rename were required in order to meet BoxesHelper expectations.
-
This method was in violation of MVC principles and its logic is now handled by the BoxesHelper.
-
This provides a cleaner view than html.erb.
-
After the refactor removing HTML generation from the Block subclass, the tests were outdated. The tests related to methods moved to the helper have been placed on its own test file and adapted accordingly since the code has changed slightly, but the assertions remained untouched (thus the refactor is correct). Finally block tests related to the view have been moved into a new view test suite.
-
Whenever calling page_trail it was pushin a new array inside the existing one while the desired behaviour is to merge them.
