Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

8/30/2024 Production Deploy #1308

Merged
merged 62 commits into from
Aug 30, 2024
Merged
Changes from 3 commits
Commits
Show all changes
62 commits
Select commit Hold shift + click to select a range
5ffab27
remove unneeded govuk dependencies
Aug 12, 2024
7b1942c
initial
Aug 19, 2024
613ee62
remove debug
Aug 19, 2024
08fe1a0
fix imports
Aug 19, 2024
0ac6404
fix tests
Aug 19, 2024
c897b2d
update page_size to 20 from 50
heyitsmebev Aug 19, 2024
686367d
Bump aiohttp from 3.9.5 to 3.10.2
dependabot[bot] Aug 19, 2024
b725bb1
Merge pull request #1260 from GSA/dependabot/pip/aiohttp-3.10.2
A-Shumway42 Aug 19, 2024
d8813a1
Bump flake8 from 7.1.0 to 7.1.1
dependabot[bot] Aug 19, 2024
791a4cc
Merge pull request #1262 from GSA/dependabot/pip/flake8-7.1.1
A-Shumway42 Aug 19, 2024
78f76df
fixed page_size testing
heyitsmebev Aug 19, 2024
19ae20a
fix tests
Aug 20, 2024
6e22cf1
fix tests
Aug 20, 2024
7acb100
fix tests
Aug 20, 2024
eca9b19
fix tests
Aug 20, 2024
d754b15
fix tests
Aug 20, 2024
e040497
fix tests
Aug 20, 2024
5c3b961
code review feedback
Aug 20, 2024
6a7717c
fix import
Aug 20, 2024
0a47481
Fix job.created_at time if necessary
Aug 20, 2024
d144cae
Merge pull request #1279 from GSA/notify-admin-1870
ccostino Aug 20, 2024
e722689
fix style checks
Aug 20, 2024
e926b74
whoops remove testing code
Aug 20, 2024
e6ea8f6
comment out repair job for now
Aug 20, 2024
74fb6b0
raise exception for big diff
Aug 20, 2024
740a45e
check test environment
Aug 20, 2024
e1ba1ac
Merge pull request #1281 from GSA/1280-change-page_size-to-20-rather-…
ccostino Aug 20, 2024
b9cfa10
Merge pull request #1284 from GSA/notify-admin-1859
ccostino Aug 20, 2024
e144662
merge from main
Aug 20, 2024
6383fee
Merge pull request #1250 from GSA/notify-api-792
ccostino Aug 21, 2024
92f1f93
Bump marshmallow from 3.21.3 to 3.22.0
dependabot[bot] Aug 21, 2024
5d5ff72
Adding the new fields for template stats.
xlorepdarkhelm Aug 21, 2024
2b1f9b7
change data retention to 7 days, 23 hours
Aug 21, 2024
030a9b5
add documentation on search tags
Aug 21, 2024
82fa52c
fix log formatting
Aug 21, 2024
6401378
update baseline
Aug 22, 2024
9b70fb9
Merge pull request #1289 from GSA/notify-api-1215
heyitsmebev Aug 22, 2024
0e33297
Merge pull request #1288 from GSA/notify-api-1287
heyitsmebev Aug 22, 2024
6d44ffc
move rotate from checks to deploy
Aug 22, 2024
05e27c7
Restructured the query to be adhering to SQLAlchemy 2.0.
xlorepdarkhelm Aug 22, 2024
ecfea7c
Merge pull request #1290 from GSA/notify-api-144
ccostino Aug 22, 2024
468b846
try Bearer instead of token
Aug 22, 2024
ba243ff
Merge pull request #1291 from GSA/notify-api-144
ccostino Aug 22, 2024
fb743cf
Fixed the joins. Now to figure out why there's no data.
xlorepdarkhelm Aug 22, 2024
9c3280f
make it possible to write secrets
Aug 22, 2024
916ba67
Merge pull request #1292 from GSA/notify-api-144
ccostino Aug 22, 2024
3e8485a
remove secrets
Aug 22, 2024
db111b4
Merge pull request #1293 from GSA/notify-api-144
heyitsmebev Aug 22, 2024
4f62d14
ugh try again with different permissions
Aug 22, 2024
861d604
add generate-salt task and remove deploy rotate-secret
Aug 23, 2024
fb23748
fix flake 8
Aug 23, 2024
91a61b1
Merge pull request #1285 from GSA/dependabot/pip/marshmallow-3.22.0
ccostino Aug 23, 2024
78d944d
code review feedback remove ability to run on higher tiers
Aug 23, 2024
aa2c8af
limit cache download by time
Aug 27, 2024
0b42364
Merge pull request #1294 from GSA/notify-api-144
terrazoon Aug 27, 2024
6a2fae5
Merge pull request #1300 from GSA/notify-api-1298
terrazoon Aug 28, 2024
4084a18
I think it is working now.
xlorepdarkhelm Aug 28, 2024
dbfd99c
Added fields to endpoint.
xlorepdarkhelm Aug 28, 2024
00e0f41
Stuff was done.
xlorepdarkhelm Aug 28, 2024
d4c4bbd
debug messages for e2e test story
Aug 28, 2024
3fd9af6
Merge pull request #1305 from GSA/notify-admin-1157
heyitsmebev Aug 28, 2024
0914cd9
Merge pull request #1304 from GSA/API-1265_adding_fields_for_template…
heyitsmebev Aug 29, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion app/config.py
Original file line number Diff line number Diff line change
@@ -84,7 +84,7 @@ class Config(object):
SQLALCHEMY_POOL_TIMEOUT = 30
SQLALCHEMY_POOL_RECYCLE = 300
SQLALCHEMY_STATEMENT_TIMEOUT = 1200
PAGE_SIZE = 50
PAGE_SIZE = 20
ccostino marked this conversation as resolved.
Show resolved Hide resolved
API_PAGE_SIZE = 250
REDIS_URL = cloud_config.redis_url
REDIS_ENABLED = getenv("REDIS_ENABLED", "1") == "1"
2 changes: 1 addition & 1 deletion tests/app/inbound_sms/test_rest.py
Original file line number Diff line number Diff line change
@@ -212,7 +212,7 @@ def test_get_inbound_sms_by_id_with_invalid_service_id_returns_404(

@pytest.mark.parametrize(
"page_given, expected_rows, has_next_link",
[(True, 10, False), (False, 50, True)],
[(True, 20, True), (False, 20, True)],
)
def test_get_most_recent_inbound_sms_for_service(
admin_request,
4 changes: 2 additions & 2 deletions tests/app/service/test_rest.py
Original file line number Diff line number Diff line change
@@ -2036,10 +2036,10 @@ def test_get_notifications_for_service_pagination_links(
resp = admin_request.get(
"service.get_all_notifications_for_service",
service_id=sample_template.service_id,
page=3,
page=6,
)

assert "?page=2" in resp["links"]["prev"]
assert "?page=5" in resp["links"]["prev"]
assert "next" not in resp["links"]