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

Feat/activate private ecs #4416

Open
wants to merge 820 commits into
base: main
Choose a base branch
from
Open

Feat/activate private ecs #4416

wants to merge 820 commits into from

Conversation

gbbayareametro
Copy link

@gbbayareametro gbbayareametro commented Oct 24, 2024 •
edited
Loading

  • Addresses the issue in full
  • Addresses only certain aspects of the issue

Description

This PR allows for the deployment and restart of the private API ECS service along with the others. It needs to be delivered along with my doorway-config PR

How Can This Be Tested/Reviewed?

The buildspec files and dev config file for the CICD codebase are the only things that changed and can be eyeballed. Other than that. See if it starts up the backend service after deploying.

Author Checklist:

  • Added QA notes to the issue with applicable URLs
  • Reviewed in a desktop view
  • Reviewed in a mobile view
  • Reviewed considering accessibility
  • Added tests covering the changes
  • Made corresponding changes to the documentation
  • Ran yarn generate:client and/or created a migration when required

Review Process:

  • Read and understand the issue
  • Ensure the author has added QA notes
  • Review the code itself from a style point of view
  • Pull the changes down locally and test that the acceptance criteria is met
  • Either (1) explicitly ask a clarifying question, (2) request changes, or (3) approve the PR, even if there are very small remaining changes, if you don't need to re-review after the updates

emilyjablonski and others added 30 commits June 5, 2024 18:24
@emilyjablonski
docs: release - updated pr template (bloom-housing#4116) (#663)
d8ae1f1
@mcgarrye
fix: demographic spoken language bugs (#642)
6fe7e2e
@KrissDrawing
fix: office hours in email template (#669)
38ea05c
@emilyjablonski
fix: release - update password requirements (bloom-housing#4107) (#654)
618e2d1
@jaredcwhite
feat: add back Font Awesome icons for create account/sign in
23750f7
@jaredcwhite
Merge branch 'main' into seeds-toast-and-icon-uptakes
1cbaaf1
@KrissDrawing
fix: use default value for accept TOS checkbox (#671)
fd86ea1
@jaredcwhite
fix: linting
ee02391
@ColinBuyck
fix: include check for address type (bloom-housing#4136) (#674)
089ea02
@jaredcwhite
fix: address various design QA for buttons (bloom-housing#4118)
c574cc4 
(cherry picked from commit 0d5f01b)
@jaredcwhite
fix: duplicate review alignment (bloom-housing#4129)
18370ae 
(cherry picked from commit b2e7778)
@ColinBuyck
fix: hide do you work question (#681)
ac79140
@ColinBuyck
fix: looser isUserSiteMatch (bloom-housing#4120) (#682)
4043cc6
@ColinBuyck
fix: fallback and refresh translations (bloom-housing#4105)
4cd0630 
* fix: fallback and refresh translations

* fix: commenting

* fix: remove logging

* fix: updated translation logic

* fix: add contentUpdatedAt field

* fix: seeding updates

* fix: add testing coverage

* fix: testing tweaks

* fix: context for new field

* fix: pr review cleanup
@ColinBuyck
fix: dwy specific testing update
18f8a1a
@ColinBuyck
fix: fallback and refresh translations (bloom-housing#4105) (#683)
1a49065
@emilyjablonski
test: release - move application page cypress tests to unit tests (bl…
83eaca8 
…oom-housing#4138) (#684)
@jaredcwhite
release: update of Seeds toast and icon (#651)
ad1bff8
@jaredcwhite
release: fix buttons and other elements from design QA (#676)
0b1ae5f
@ludtkemorgan
fix: allow removal of mailing address (bloom-housing#4130)
495b9da
@ludtkemorgan
fix: add error message for incorrect listing approval (bloom-housing#…
610b19a 
…4134)
@ludtkemorgan
fix: toast test fix
3d0b5b3
@ludtkemorgan
fix: allow removal of mailing address (#680)
d441cff
@ludtkemorgan
fix: add msw package to public (#685)
c6f57da
@emilyjablonski
fix: release - seed data and account profile saving issues (#670)
b07a024
@ludtkemorgan
fix: remove last reference to backend in docker (#686)
d337932
@ludtkemorgan
fix: one more backend dependency removed (#687)
02d406a
@ludtkemorgan
fix: add babel dependency (#688)
4fce094
@jaredcwhite
fix: clearing out min/max rent filter
e1254fb
@ColinBuyck
fix: add language to headers (bloom-housing#4148) (#693)
229adef
ColinBuyck and others added 28 commits October 2, 2024 16:32
@ColinBuyck
fix: extend to three hours (#869)
039cfb3
@jaredcwhite
feat: add various content updates (#842)
2ace110
@jaredcwhite
fix: convert single quote entities in Opportunities email (#854)
25e135d
@ludtkemorgan
fix: add user and application import scripts (#759)
faedb35
@ludtkemorgan
fix: remove backend core directory (bloom-housing#4344) (#856)
60178b9
@ColinBuyck
fix: match pref on id (bloom-housing#4372) (#878)
fe72b40
@jaredcwhite
fix: household members link on application review (bloom-housing#4385)
4460dfa 
* also add reusable layout footer to the household members page

(cherry picked from commit 29abe89)
@mcgarrye
release: record of listing copy (bloom-housing#4375) (#884)
965dee6
@emilyjablonski
fix: dont attempt to send email to no users (bloom-housing#4352)
cf01c59
@jaredcwhite
fix: household members link on application review (bloom-housing#4385) (
59c6a98 
#881)
@YazeedLoonat
fix: may the force be with you (bloom-housing#4388)
9e559f3
@ludtkemorgan
fix: add required field for applications in script (#882)
b03e937
@emilyjablonski
chore: depcheck and missing deps - doorway uic (#880)
54b6964
@gbbayareametro @ludtkemorgan
feat: adding additional admin users to dev seed for performance testi…
11697e8 
…ng … (#898)

Co-authored-by: Morgan Ludtke <[email protected]>
@ludtkemorgan
fix: script runner for new reserved community type (bloom-housing#4380)…
ebe8050 
… (#886)
@mcgarrye
release: flaky backend tests (bloom-housing#4389) (#901)
3cee086
@emilyjablonski
refactor: remove unused doorway uic components (#891)
1412e5e
@YazeedLoonat
fix: if programs is null no longer errors (bloom-housing#4362)
2294388 
* fix: if programs is null no longer errors

* fix: moving some stuff around to get test cov up
@YazeedLoonat
fix: moving test to e2e
d5d7552
@YazeedLoonat
fix: using correct logger
fc84416
@YazeedLoonat
feat: exposes pref unit type (bloom-housing#4401)
984f6f5
@emilyjablonski
refactor: postcss vulnerability and cleanup (#904)
d9c2722
@KrissDrawing
fix: change begin to language name (#872)
936c0a6
@YazeedLoonat
fix: exposing more closed listings (bloom-housing#4402)
4f9261d 
* fix: exposing more closed listings

* fix: update to use a new env variable
@ColinBuyck
fix: updated email and logic for single use code (bloom-housing#4376) (…
9bcd3d7 
…#908)
@ludtkemorgan
fix: cleanup lint warnings (bloom-housing#4404) (#900)
f5ddec2
@mcgarrye
release: add farmworkerHousing and update other translations (#913)
722ad4f
@gbbayareametro
feat: adding private backend service
4cc8eb4
@gbbayareametro
Copy link
Author

Wrong repo. feel free to reject (I cant get rid of it apparently)

@ludtkemorgan
Copy link
Collaborator

@gbbayareametro I think this PR was accidentally opened up against Bloom Core instead of Doorway. For some reason Github defaults to that when creating a PR.

I'm assuming this is still a valid change, but unfortunately you'll need to close this one and reopen it in Doorway

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@gbbayareametro @ludtkemorgan @emilyjablonski @mcgarrye @KrissDrawing @jaredcwhite @ColinBuyck @YazeedLoonat