Skip to content
This repository has been archived by the owner on Nov 14, 2023. It is now read-only.

Including amenities and places to hashtag endpoint #178

Merged
merged 4 commits into from
May 18, 2022

Conversation

ramyaragupathy
Copy link
Member

Modified the base model, query builder and test function to include new amenities and places statistics.

cc @itskshitiz321

@@ -427,7 +427,8 @@ def test_organization_hashtag_weekly_query():
where name = 'msf'
),
t2 as (
select name as hashtag, type as frequency , start_date , end_date , total_new_buildings , total_uq_contributors as total_unique_contributors , total_new_road_m as total_new_road_meters
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update will be needed on Assertion Step ! Replacement of existing query with updated query that will be generated from query builder

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed through 3e963b3 and 3c24122

@@ -455,7 +456,8 @@ def test_organization_hashtag_monthly_query():
where name = 'msf'
),
t2 as (
select name as hashtag, type as frequency , start_date , end_date , total_new_buildings , total_uq_contributors as total_unique_contributors , total_new_road_m as total_new_road_meters
select name as hashtag, type as frequency , start_date , end_date , total_new_buildings , total_uq_contributors as total_unique_contributors , total_new_road_m as total_new_road_meters,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed through 3e963b3 and 3c24122

@ramyaragupathy
Copy link
Member Author

@itskshitiz321 - please check now

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants