Skip to content

Commit 44f96ab

Browse files
Merge pull request #14 from NYPL/ignore-qa-new-locations
Don't log a warning for new locations in QA
2 parents 5cf7ade + 9ad76fb commit 44f96ab

File tree

3 files changed

+7
-1
lines changed

3 files changed

+7
-1
lines changed

.python-version

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
3.9.16

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
1+
## 2023-11-17 -- v1.0.3
2+
### Fixed
3+
- Do not log a warning for new locations in QA
4+
15
## 2023-06-28 -- v1.0.2
26
### Fixed
37
- Updated `configure-aws-credentials` GitHub action version

main.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,8 @@ def poll_location_hours(logger):
7272
lambda daily_hours: daily_hours['day'] == api_weekday,
7373
location['hours']['regular']))
7474
redshift_hours = redshift_dict.get(location['id'], None)
75-
if redshift_hours is None:
75+
if (redshift_hours is None
76+
and os.environ['ENVIRONMENT'] == 'production'):
7677
logger.warning('New location id found: {}'.format(location['id']))
7778

7879
# Check today's regular hours in Redshift against today's regular hours

0 commit comments

Comments
 (0)