Skip to content

Commit 74bc05a

Browse files
Merge pull request #85 from OpenSourcePolitics/managed_users_in_users_model
feat(demo): count managed non-confirmed users in users model
2 parents 4a919ae + aecf271 commit 74bc05a

File tree

3 files changed

+20
-4
lines changed

3 files changed

+20
-4
lines changed

projects/demo/models/marts/users/schema.yml

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,23 @@ unit_tests:
124124
expect:
125125
format: dict
126126
rows:
127-
- {id: 2}
127+
- {id: 2}
128+
- name: managed_and_confirmed_users_check
129+
description: "Check that only managed users are retrieved among unconfirmed users"
130+
model: users
131+
given:
132+
- input: ref('all_users')
133+
format: dict
134+
rows:
135+
- {id: 1, deleted_at: NULL, blocked: false, admin: false, managed: false, confirmed: true}
136+
- {id: 2, deleted_at: NULL, blocked: false, admin: false, managed: true, confirmed: false}
137+
- {id: 3, deleted_at: NULL, blocked: false, admin: false, managed: false, confirmed: false}
138+
expect:
139+
format: dict
140+
rows:
141+
- {id: 1}
142+
- {id: 2}
143+
128144
# - name: test_url_concatenation
129145
# description: "Check URL concatenation logic"
130146
# model: all_users

projects/demo/models/marts/users/users.sql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,4 +51,4 @@ SELECT
5151
WHERE deleted_at IS NULL
5252
AND decidim_users.blocked != true
5353
AND decidim_users.admin != true
54-
AND decidim_users.confirmed = true
54+
AND (decidim_users.confirmed = true OR decidim_users.managed = true)

projects/marseille/models/marts/users_with_promote_info.sql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ WITH promote_actions AS (
55
decidim_action_logs.id AS log_id,
66
action
77
FROM {{ ref('stg_decidim_action_logs') }} AS decidim_action_logs
8-
LEFT JOIN {{ ref('all_users') }} AS users ON users.id = decidim_action_logs.resource_id
8+
LEFT JOIN {{ ref('users') }} AS users ON users.id = decidim_action_logs.resource_id
99
WHERE resource_type = 'Decidim::User'
1010
AND action = 'promote'
1111
)
@@ -18,5 +18,5 @@ SELECT
1818
WHEN promote_actions.action IS NULL AND users.managed = false THEN 'Utilisateur non représenté'
1919
ELSE 'Non proposée'
2020
END) AS promotion_status
21-
FROM {{ ref('all_users') }} AS users
21+
FROM {{ ref('users') }} AS users
2222
LEFT JOIN promote_actions ON users.id = promote_actions.user_id

0 commit comments

Comments
 (0)