08 Jun, 2016
4 commits
-
This reverts commit 5a337d1a3abfc8f9d789ec8efd6e41575a416f54. The right way to set the subdirectory is by setting the Rack mount on config.ru or on the webserver like apache or nginx.
-
Signed-off-by: Heitor Reis Ribeiro <marcheing@gmail.com>
20 Apr, 2016
1 commit
-
There were instance variables filled by fetching the latest content every time, defeating the purpose of the fragment caches in the view. Replace them with helper methods, that also make the number of items visible in the view. Signed-off-by: otaviocv <otavio@deluqui.com.br>
18 Apr, 2016
1 commit
16 Apr, 2016
1 commit
-
It makes it harder for an user to mistakenly add a metric to their script that will never work. Signed-off-by: Eduardo Araújo <duduktamg@hotmail.com>
15 Apr, 2016
1 commit
-
They were moved to mezuro.github.io Signed-off-by: Rafael Reggiani Manzo <rr.manzo@protonmail.com>
13 Apr, 2016
1 commit
-
The acceptance test has been refactored accordingly and got more extended to untested features.
11 Apr, 2016
1 commit
-
Ensure webhook requests are only accepted after proper verification of expected headers, repository address and branch, using the newly implemented library. To that effect, split up the logic to extract that information from requests to it's own class, and use it in the notify_push action. Also, add factories to make better testing of the notify_push action possible, such as a factory similar to a real request from Gitlab, and one for the Kalibro Client repository on Gitlab. Adding other hook providers should be quite a lot easier after this is applied.
04 Apr, 2016
1 commit
-
Since it's a remote request there's no CSRF token to be sent, so don't require one.
30 Mar, 2016
1 commit
-
Update KalibroClient to version v4.0.0.alpha1 Signed-off-by: Daniel Miranda <danielkza2@gmail.com>
17 Feb, 2016
1 commit
-
Now scope has a hash with key type.
29 Jan, 2016
1 commit
-
This is expected to improve compatibility with reverse-proxies. Still, for full compatibility, regarding redirections, we still may need to set the default url options with the proxy host. Signed-off-by: Rafael Reggiani Manzo <rr.manzo@gmail.com>
22 Jan, 2016
3 commits
-
Calling methods with self improve the readability as it makes clear that it is not a variable but a method! Signed off by: Daniel Miranda <danielkza2@gmail.com>
-
It will never return nil. At most, it will raise a ActiveRecord::RecordNotFound Signed off by: Diego Araújo <diegoamc90@gmail.com>
-
Signed off by: Diego Araújo <diegoamc90@gmail.com>
20 Jan, 2016
6 commits
-
Signed-off-by: Rafael Reggiani Manzo <rr.manzo@gmail.com>
-
The code is pretty straightforward, just raising an Error in order to force subclasses to implement this. Signed-off-by: Rafael Reggiani Manzo <rr.manzo@gmail.com>
-
Also improve error reporting functions
-
Move as much code as possible to BaseMetricConfigurationsController instead of repeating it ad-hoc in the concrete classes. Most of the concrete code has gone away except for some small overrides: - Hotspot's does not have a 'new' action, and therefore can't redirect to it. It also does not accept any MetricConfiguration parameters. - Metric's only accepts 'weight' and 'aggregation_form' MetricConfiguration parameters. It also needs to fetch ReadingGroups to display in the 'choose_metric' action. - Compound's needs to force the Metric type when creating the metric, and must accept parameters for Metric ('name', 'description', 'script', 'scope', 'code') in addition to MetricConfiguration's 'weight'. Additionally: - Update cache clearing to clear caches for Hotspot and Tree configurations - Verify the existence of the KalibroConfiguration and use the object that was fetched whenever necessary, instead of only using the id. - Ensure the KalibroConfiguration and ReadingGroup ids match between the values inside and outside the 'metric_configuration' parameter. Previously it was possible to send one in the URL and update another, which might even have been a security issue. - Allow sending either 'metric_code' or 'metric_name' as a parameter. Both are used in different places in the code, but making them uniform is a job for a later time. Signed off by: Daniel Miranda <danielkza2@gmail.com>
-
Now it doesn't redirect to the tree metric configuration creation page after choosing a hotspot metric configuration Adds a bunch of code from removed commit ec4c3ef6f29aa34f3c58819df4f84725aff8bd60 that fixed the view after the modification introduced by 39bc97f568d6989ae98b1f5fd5452515ee54045f. It fixes the view to properly work with the path helper. This is a squash of two commits. Signed off by: Diego Araújo <diegoamc90@gmail.com> Signed off by: Heitor Reis <marcheing@gmail.com>
-
It has just the prototype for the method so its route works. This route will be useful when creating the upcoming helper method for metric configuration creation form. Signed off by: Heitor Reis <marcheing@gmail.com>
12 Jan, 2016
1 commit
-
Using URL helpers is not friendly with proxies like Colab. The path helpers have the same effect unless we want sometime to render pages outside Prezento context like embeds or sending emails. Signed off by: Diego Araújo <diegoamc90@gmail.com>
23 Nov, 2015
2 commits
-
Signed off by: Daniel Miranda <danielkza2@gmail.com>
-
Gitlab sends the 'X-Gitlab-Event' header for all the webhook requests. Verify it so we reduce the chance that robots call the hooks accidentally. Signed off by: Daniel Miranda <danielkza2@gmail.com>
19 Nov, 2015
1 commit
-
* Includes this action in the list of actions that do not need user authentication. Signed-off-by: Heitor Reis <marcheing@gmail.com>
09 Nov, 2015
1 commit
-
* Renamed ProjectAttributes' column hidden to public and inverted default values * Implemented Project#"public_or_owned_by_user" * All features now create ProjectAttributes after creating a Project * Implemented ProjectAttributes#project and ProjectAttributes#project= * Fix Project#attributes to return nil when there is no ProjectAttribute * Fixes #263 Signed off by: Diego Araújo <diegoamc90@gmail.com>
28 Oct, 2015
1 commit
-
Signed off by: Eduardo Araújo <duduktamg@hotmail.com>
26 Oct, 2015
3 commits
-
The search and replace missed this. Now when a new CompoundMetriConfiguration gets created or updated the KalibroConfiguration show page gets a clear cache. Signed off by: Heitor Reis <marcheing@gmail.com>
-
Put a blank space between # and text and don't forget about tagging pending fixes with FIXME. Signed off by: Eduardo Araújo <duduktamg@hotmail.com>
-
Signed off by: Eduardo Silva Araújo <duduktamg@hotmail.com> Signed off by: Diego Araújo <diegoamc90@gmail.com>
11 Sep, 2015
1 commit
14 Aug, 2015
1 commit
08 Jul, 2015
2 commits
-
Fixed acceptance tests and removed project list selector from repository form Signed-off-by: Pedro Scocco <pedroscocco@gmail.com> Signed off by: Diego Araújo <diegoamc90@gmail.com>
-
Signed-off-by: Daniel Miranda <danielkza2@gmail.com>
06 Jul, 2015
1 commit
-
Signed off by: Rafael Reggiani Manzo <rr.manzo@gmail.com>
24 Jun, 2015
1 commit
-
Signed off by: Daniel Miranda <danielkza2@gmail.com> Signed-off-by: Pedro Scocco <pedroscocco@gmail.com> Signed off by: Heitor Reis <marcheing@gmail.com>
22 Jun, 2015
1 commit
-
Actions new and create check if project_id was provided before trying to check the project ownership
17 Jun, 2015
1 commit
-
new and create action can be acessed both with or without a project. This enables to have repositories the two ways. The behaviour for the back had to change from going back to a specific project page. To the all projects pages since some repositories may not have a project_id. With the routes, many helpers had to get fixed together with acceptance tests steps (for this a new step was created in order to be able to create RepositoryAttributes). Finally the JS for repository state retrieval needed fixes since it builds the route withot any helper. Same thing for the JS that prevents infinite state queries. Signed off by: Rafael Reggiani Manzo <rr.manzo@gmail.com>