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.

Sign up for GitHub

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

Jump to bottom

Release 2023.08.1 #300

Merged
merged 15 commits into from
Aug 16, 2023
Merged

Release 2023.08.1 #300

merged 15 commits into from
Aug 16, 2023

Conversation

thekaveman
Copy link
Member

Closes #283

@thekaveman
chore(devcontainer): update customizations syntax
26489b3
@thekaveman
chore(devcontainer): normalize settings
f8ba8d5 
with the benefits devcontainer
@thekaveman
Chore: small devcontainer improvements (#277)
dc4e512
@thekaveman
chore(data): add header row
5eadec5 
test_init_db_command fails with user count 26 != asserted 25
@thekaveman
fix(db): use csv.DictReader to parse header row
b4c398a
@thekaveman
Fix: skip CSV header on data load (#279)
82f0364
@thekaveman
refactor: use pyproject.toml exclusively
cc935bf 
move version to within app __init__
@thekaveman
refactor(docker): update build to install pyproject.toml
9982e7d
@thekaveman
chore(github): update workflows and dependabot
3880b7e
@thekaveman
chore(terraform): route uptime check through Front Door
f7c2ad1 
more realistic and end-to-end test of the infrastructure and app
@thekaveman
Chore: route uptime check through FrontDoor (#281)
a06c096
@thekaveman
Refactor: pyproject.toml (#280)
ce3acab
@thekaveman
chore: bump version for release
ba65714
@thekaveman
Chore: bump version for release (#298)
3025632
@thekaveman
Deploy to test (#299)
68df688
@thekaveman thekaveman self-assigned this Aug 16, 2023
@thekaveman thekaveman requested a review from a team as a code owner August 16, 2023 22:34
@thekaveman thekaveman marked this pull request as draft August 16, 2023 22:34
@thekaveman thekaveman marked this pull request as ready for review August 16, 2023 22:36
@thekaveman thekaveman linked an issue Aug 16, 2023 that may be closed by this pull request
Release to production #283
Closed
@thekaveman
Copy link
Member Author

Confirmed the MST Courtesy Card flow in test

machikoyasuda
machikoyasuda approved these changes Aug 16, 2023
View reviewed changes
Copy link
Member

@machikoyasuda machikoyasuda left a comment

Choose a reason for hiding this comment

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

Confirmed happy path, unhappy path for MST/CC on test

@thekaveman thekaveman merged commit 89a9bfc into prod Aug 16, 2023
7 checks passed
@angela-tran
Copy link
Member

Should we create a tag and GitHub release for this?

@thekaveman
Copy link
Member Author

@angela-tran Yeah, it was on my list but the bugfix took priority.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@machikoyasuda machikoyasuda machikoyasuda approved these changes

Assignees

@thekaveman thekaveman

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Release to production
3 participants
@thekaveman @angela-tran @machikoyasuda