consul/nairobi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
8,984 Commits 1 Branch 0 Tags
9a217348051c2fffc988e149dc99ca2a32a51c0c
Commit Graph

8984 Commits

This Branch
This Branch
All Branches
Author SHA1 Message Date
Alberto Calderón Queimadelos
9747386575 Merge pull request #2506 from consul/fix_email_spec_comment_failues 
Fix Email Spec comment random failures
 2018-03-02 15:06:32 +01:00
iagirre
e21588ec16 The function comment_on has been modified to change the functionality. Now, a comment is generated and a email notification send manually, so that JS has been avoided, that was the original problem. 2018-03-02 13:35:50 +01:00
Bertocq
316c0aab3e Remove unnecesary slug generation trigger 2018-03-02 09:17:34 +01:00
Bertocq
16bec030ad Cleanup spec files from I18n calls 2018-03-02 09:17:29 +01:00
Bertocq
d9a8474793 Refactor Group partial to be easier to read 2018-03-02 09:17:26 +01:00
Bertocq
7c02f9c82e Fix missing I18n.t usage 2018-03-02 09:17:21 +01:00
Raúl Fuentes
78fc4010d0 fixed the style 2018-03-02 09:17:08 +01:00
Raúl Fuentes
8f4d48690b Added tests for edit the name of any Budget::Group 2018-03-02 09:17:03 +01:00
Raúl Fuentes
6d8d44a5e3 Added interface and methods for edit the name of a Budget::Group if the budget is still inthe drafting phase 2018-03-02 09:16:57 +01:00
Bertocq
6b41b6487b Add UserSegments#user_segment_emails helper method 
Why:

Both Newsletters and Email Downloads need the same logic: To extract the
emails from all the users in the segment that have newsletter flag
active, removing all empty email values.

How:

1- UserSegments#user_segment_emails holds that repeated logic and is used
on both Newsletter & EmailDownload.

2- Rename Newsletter#list_of_recipients to list_of_recipient_emails as
it is more descriptive. There is no need to pass entire Users around,
only the emails are needed at Mailer#newsletter method.

3- Cleanup Newsletter#list_of_recipient_emails model spec scenario
 2018-03-01 20:59:20 +01:00
Bertocq
99851e9588 Remove empty emails from users segments recipients 
Why:

User with an empty email value (nil) should not appear in the recipient
list for a given UserSegment at Newsletters or Email Downloads.

How:

Using Enumerable#compact and Enumerable#select to filter out empty emails

Increasing Email Download feature spec and Newsletter model spec to cover
all possible scenarios including the nil email one.
 2018-03-01 20:09:32 +01:00
Bertocq
089cef8ead Improve download segment translations 2018-03-01 13:52:48 +01:00
Alberto Calderón Queimadelos
0decaf43a3 Merge pull request #2492 from consul/fix_newsletter_send_performance 
Fixed how newsletters controller and mailer handle recipients
 2018-02-28 19:03:58 +01:00
Bertocq
61c58817fc Update unreleased section of changelog 2018-02-28 18:09:46 +01:00
Alberto Calderón Queimadelos
4af603edc6 Merge pull request #2491 from consul/2487-feasible_undecided_investment_authors_segment 
Fix UserSegment feasible and undecided investment authors
 2018-02-28 18:01:54 +01:00
Alberto Calderón Queimadelos
0acb4988f0 Merge pull request #2490 from consul/documents-tab 
Documents list
 2018-02-28 17:14:58 +01:00
María Checa
3867c6b716 Fixed newsletter tests 2018-02-28 17:02:39 +01:00
María Checa
8d2a103744 Changed how newsletters controller and mailer handle recipients 
Now newsletters controller calls the Mailer method to send a newsletter once per user.
 2018-02-28 17:02:31 +01:00
Alberto Calderón Queimadelos
494904565a Merge pull request #2489 from consul/ui-fixes 
UI fixes
 2018-02-28 16:56:57 +01:00
María Checa
6140a822e8 Changed translations 2018-02-28 16:55:53 +01:00
María Checa
7913128520 Changed tests for UserSegments#feasible_and_undecided_investment_authors 2018-02-28 16:54:28 +01:00
María Checa
678ea0bef2 Added unfeasible investments with valuation not finished to UserSegments#feasible_and_undecided_investment_authors 2018-02-28 16:54:28 +01:00
decabeza
53e36850a4 Fixes documentable rubocop offences 2018-02-28 16:21:22 +01:00
decabeza
e79f64c009 Updates specs 2018-02-28 16:21:18 +01:00
decabeza
1546f67367 Removes document tab on legislation proposals 2018-02-28 16:21:08 +01:00
decabeza
7a888b71fd Removes duplicate heading info, fixes headings structure 2018-02-28 16:10:14 +01:00
decabeza
46f21330bd Removes document tab on budgets investments 2018-02-28 15:23:39 +01:00
decabeza
5b7a5a2b4b Removes document tab on proposals 2018-02-28 15:23:32 +01:00
decabeza
466f8ef4e0 Removes unnecessary css 2018-02-28 15:23:22 +01:00
decabeza
5c0a10699a Refactors documents partials and reorganices css 2018-02-28 15:22:33 +01:00
decabeza
bd3929a7dc Adds missing translation for legislation filter title 2018-02-28 15:19:44 +01:00
decabeza
ddc4ff329b Fixes button vertical align on polls list 2018-02-28 15:19:33 +01:00
Bertocq
c2e9096ce1 Improved user segments en & es translations 2018-02-26 16:56:28 +01:00
Alberto Calderón Queimadelos
f5f37829e0 Merge pull request #2486 from wairbut-m2c/docs/nodejs-requirement 
docs(readme): Include Node.js as requirement
 2018-02-26 16:17:41 +01:00
Angel Perez
9443e585c6 docs(readme): Include Node.js as requirement 
Failed CONSUL installation reported on #2391 because
Node.js was missing on the development environment
 2018-02-26 10:33:22 -04:00
Bertocq
dbe723079e Disable guide page when Budget is not accepting 
Why:

When there is not Budget accepting (Investment creation) the guide page
doesn't have much sense as it will give the user an option that can't be
used (creating an Investment).

How:

Using `Budget.current&.accepting?` conditional at GuidesHelper to link
to new proposal link instead of guide page, and adding an scenario to
guides feature spec for it.
 2018-02-26 10:01:30 +01:00
Bertocq
c3a3e76304 Improve newsletter model spec list_of_recipients 2018-02-22 18:24:36 +01:00
Bertocq
e3eeca4f06 Improve newsletter model spec with valid_segment_recipient? 2018-02-22 18:24:18 +01:00
Bertocq
d44621f761 Relocate spending proposal valuation assigment seed 2018-02-22 11:38:49 +01:00
Bertocq
500db4b5d8 Add translations used on dev seeds files 2018-02-22 11:04:49 +01:00
Bertocq
54e6c5fc5c Split dev_seeds into individual files for sections 
Why:

Its a really huge script, and conflicts are hard to resolve on forks,
with indivudal scripts its easier to make custom changes.

How:

Following @mariacheca example using require_relative and a file under
the db/dev_seeds/ folder
 2018-02-22 11:04:47 +01:00
Bertocq
7d88b25741 Revert "Revert default map change" 
This reverts commit 418dc6a1d6.
 2018-02-22 07:40:59 +01:00
Bertocq
65288573b5 Fix user administrators scope relation 2018-02-22 00:42:40 +01:00
Bertocq
61f015e2c2 Add administrators UserSegment 
We need a user segment to have only administrators as recipients of both
newsletters and notifications
 2018-02-21 23:34:20 +01:00
María Checa
4bf20e5bd9 Removed rubyzip gem from Gemfile.lock 2018-02-21 16:06:55 +01:00
Bertocq
c572fe26e9 Update unreleased section of changelog 2018-02-21 13:04:57 +01:00
Alberto Calderón Queimadelos
df16194c78 Merge pull request #2474 from consul/admin_newsletter_email_refactor 
Admin newsletter email refactor
 2018-02-21 13:03:35 +01:00
Bertocq
12fd0a7cd3 Make newsletter seed file line lenght compliant 2018-02-21 11:47:34 +01:00
Bertocq
fb2c92228a Refactor UserSegment class 
Simple refactor to avoid creating unnecessary variables and make it easier
to read.
 2018-02-21 11:47:30 +01:00
Bertocq
24aa157998 Fix invalid Newsletter recipients count to 0 
When a newsletter doesn't have a valid user segment for the recipients
list, the number of users should be 0.
 2018-02-21 11:47:24 +01:00