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

Prod Deploy #2285

Open
wants to merge 76 commits into
base: production
Choose a base branch
from
Open

Prod Deploy #2285

wants to merge 76 commits into from

Conversation

alexjanousekGSA
Copy link
Contributor

@alexjanousekGSA alexjanousekGSA commented Jan 21, 2025 •
edited by ccostino
Loading

🚨 DO NOT MERGE YET 🚨

We cannot merge this yet, even after our own reviews are finished, until additional content review has taken place.

Description

This release is for a minor copy update with a few minor changes

Release notes can be found here

Security Considerations

  • The commits included highlighted one setting that allowed admins to push to main without a PR. This has been updated and is now prevented

Post Deploy Instructions:

The feature flag may still be enabled for the prod environment. This command will update demo/pro

cf login -a api.fr.cloud.gov --sso
cf set-env notify-admin-prod FEATURE_ABOUT_PAGE_ENABLED true
cf restage notify-admin-prod

Previously:

image

Updated Text:

image

heyitsmebev and others added 30 commits December 12, 2024 18:42
@heyitsmebev
added id and also refactor error message on templates page
17f758b
@heyitsmebev
adding error annoucement
b731cd6
@heyitsmebev
adding error annoucement
12636d6
@heyitsmebev
adding id
56381f5
@heyitsmebev
Merge branch 'main' into 2125-send-message-a11y-audit-unique-ids
ca5e5ce
@heyitsmebev
adding required html field
1df019e
@heyitsmebev
fixed testing
1e261fd
@heyitsmebev
first round
a3db0a9
@heyitsmebev
draft dash
9d09f75
@heyitsmebev
draft dash
30293d9
@heyitsmebev
clean up
6d008d4
@heyitsmebev
clean up
ce87e21
@heyitsmebev
clean up
2159a9d
@heyitsmebev
poetry
5c0eb91
@heyitsmebev
Merge branch 'main' into 2199-add-pending-message-data-to-daily-and-u…
82179c0 
…ser_daily-stats
@heyitsmebev
jest test
5e61fb8
@heyitsmebev
fixed drop down
cd90682
@heyitsmebev
updated fetching interval
607e112
@heyitsmebev
updating job status states
8754c03
@heyitsmebev
update banner with different status states
ae083f4
get rid of some noise in status updates
3e8015e
fix test
6f7202e
fix test
1c77f8d
@heyitsmebev
changed pending state
c6ed4a8
@heyitsmebev
update verbage
7c51799
@heyitsmebev
refactor code
dfe421b
@heyitsmebev
refactor code partial
79e8f02
@heyitsmebev
remove service
839bd0b
@heyitsmebev
Reformat
7422b61
@heyitsmebev
ajax applied to status
9942ecf
@alexjanousekGSA
Merge pull request #2295 from GSA/dependabot/npm_and_yarn/babel/prese…
088da55 
…t-env-7.26.7

Bump @babel/preset-env from 7.26.0 to 7.26.7
@heyitsmebev
Merge pull request #2250 from GSA/2199-add-pending-message-data-to-da…
6c3d6ee 
…ily-and-user_daily-stats

Add pending to dashboard
@sheevdave
Merge pull request #2292 from GSA/sheevdave-patch-2
3e37915 
Update pull_request_template.md
@stvnrlly stvnrlly deployed to production January 30, 2025 12:05 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@heyitsmebev heyitsmebev heyitsmebev left review comments

@ccostino ccostino Awaiting requested review from ccostino

@terrazoon terrazoon Awaiting requested review from terrazoon

@stvnrlly stvnrlly Awaiting requested review from stvnrlly

@xlorepdarkhelm xlorepdarkhelm Awaiting requested review from xlorepdarkhelm

@A-Shumway42 A-Shumway42 Awaiting requested review from A-Shumway42

@jonathanbobel jonathanbobel Awaiting requested review from jonathanbobel

At least 2 approving reviews are required to merge this pull request.

Assignees

@alexjanousekGSA alexjanousekGSA

Labels
None yet
Projects
Notify.gov product board
Status: Issue Backlog (More than 3 Months)
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@alexjanousekGSA @ccostino @heyitsmebev @jonathanbobel @stvnrlly @sheevdave @terrazoon