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

Fhb 1260 #492

Draft
wants to merge 82 commits into
base: release-merge-find-and-connect
Choose a base branch
from
Draft

Fhb 1260 #492

wants to merge 82 commits into from

Conversation

tgohil-hippo
Copy link
Contributor

Description

Refactored the Single directory e2e tests to include file structure ahead of adding filters tests.

Checklist

  • Jira ticket ID included in the title (unless not applicable)
  • Short, descriptive and sentence-case title used briefly describing the change
  • Commit messages are short, informative and accurate. When in doubt, use Conventional Commits
  • Documentation related to the change has been added/updated as appropriate
  • Changes have been given a once-over for spelling and grammar errors
  • Tests have been added to cover any changes

@tgohil-hippo tgohil-hippo changed the base branch from main to release-merge-find-and-connect March 11, 2025 09:50
Zac-Digital and others added 29 commits March 11, 2025 09:53
@Zac-Digital
feat: add feature flag management functionality
8c76292
@Zac-Digital
feat: navigation links are feature flagged
2314ad0
@Zac-Digital
chore: remove commented code
bcb0e1c
@Zac-Digital
feat: getting list of vcfs services gated behind feature flag
90d1142
@Zac-Digital
feat: non-authenticated journey, no service type discrimination
6693907
@Zac-Digital
refactor: revert la services as its part of a different ticket
f72a6ce
@Zac-Digital
chore: remove unused import
804380c
@Zac-Digital
chore: update comment
a8906c3
@Zac-Digital
chore: update comment with relevant ticket
62a5e1e
@stevesatdfe
feat: Redirect find to connect
0c30fa6
@stevesatdfe
feat: Add postcode search redirect
dd29380
@stevesatdfe
Add query parameters
5a127b7
@stevesatdfe
Extract query parameters explicitly
7c9658d
@stevesatdfe
Redirect search
d4c028d
@stevesatdfe
Try temp disable of visit record
631621a
@stevesatdfe
Try temp record fix
f079da1
@stevesatdfe
feat: Add organisation name to the service DTO for displaying
9fd10cc
@Zac-Digital
wip: abomination
c3a52dd
@Zac-Digital
wip: styling
f34bf36
@Zac-Digital
refactor: move filters into partial views
55c1269
@Zac-Digital
wip: checkboxes stay checked
a9947a5
@Zac-Digital
wip: remove unused stuff
72fac3f
@Zac-Digital
refactor: clean up frontend code
0162850
@Zac-Digital
refactor: remove unused stuff
ef0119a
@Zac-Digital
chore: create todo list
a58b1aa
@Zac-Digital
refactor: put the languages back
813759c
@Zac-Digital
refactor: ui alignment
5480535
@Zac-Digital
refactor: frontend tomfoolery
2fa2feb
@Zac-Digital
chore: remove leftover dashboard bits
cd1a16d
DamiLaleye1 and others added 29 commits March 11, 2025 09:58
@DamiLaleye1
added description assertion
6d19e5d
@DamiLaleye1
update service area css tag
4187c29
@DamiLaleye1
updated css locators
9d7e844
@DamiLaleye1
refactored actor names to actor-user-role file
08c0140
@Zac-Digital
refactor: remove .env and fix .env.example
07c8b54
@stevesatdfe
feat(terraform): Add a workbook dashboard to visualize our services w…
22f979c 
…ith environment configuration
@stevesatdfe
feat(terraform): Add ADF to dashboard template
e6df325
@stevesatdfe
chore(terraform): Update log retention to match GDS guidance
4838b4b
@armstrong-pv
Initial Terraform folder restructure
46e8095
@armstrong-pv
Incorporate new slack support email address (secrets added to Github)
10a5881
@armstrong-pv
Added missing stack variable for support email
83366ad
@armstrong-pv
Renamed slack channel email
610ac43
@armstrong-pv
Separated existing alerts into alerting.tf and added app gateway alerts
10945bb
@armstrong-pv
Fix resource group name
73de69a
@armstrong-pv
Fix metric namespace for storage acount
433cb43
@armstrong-pv
Test fixed name for alerts resource group based on environment name
2bf1f52
@armstrong-pv
Revert "Test fixed name for alerts resource group based on environmen…
13f25ee 
…t name"

This reverts commit af1e556.
@armstrong-pv
Add app service plan alerts
36aaafd
@armstrong-pv
Added app service alerts (web / function apps)
4570399
@armstrong-pv
Fix: Time aggregation must be one of [Count, Total, Minimum, Maximum]…
6240f77 
… (average not supported)
@armstrong-pv
Exclude function apps from CPUTime alert - they don't support this me…
c5828b3 
…tric
@armstrong-pv
Use lookup rather than coalesce when specifying optional attributes
2d60371
@armstrong-pv
Add ADF failed pipeline runs alert
a2d8b5e
@armstrong-pv
Removed alerts for the open_referral_mock_api_web_app and fh_idam_mai…
bc7d517 
…ntenance_ui app services after discussion with Dami, Aaron and Tina. They are usually switched off and the mock api is for internal testing.
@armstrong-pv
Fix the ADF alerts to treat the ADF as an optional environment compon…
88a43a6 
…ent. ADF doesn't exist for Test2.
@armstrong-pv
Removed the alerts for the open_referral_function_app
2a6856e
@armstrong-pv
Pin terraform version to 4.19.0 and fix deprecation warnings
ee91444
resolved merge conflicts
14f6b5e
tests: added file structure to single directory tests
7824917
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.
6 participants
@tgohil-hippo @Zac-Digital @stevesatdfe @Top-Cat @DamiLaleye1 @armstrong-pv