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

Pathways content handling #606

Merged
merged 92 commits into from
Jan 10, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
92 commits
Select commit Hold shift + click to select a range
ec8982e
plumbing for pathways content types
mattb-hippo Dec 4, 2024
aa7b79a
Add dependabot.yml (#575)
as2024ad Dec 5, 2024
5d8cc8d
On trigger.
as2024ad Dec 6, 2024
28fe546
build(deps): bump cross-spawn (#571)
as2024ad Dec 6, 2024
340cb2a
created navigation helpers
mattb-hippo Dec 6, 2024
363a86c
WiP: pathways module and module section
Dec 9, 2024
4d0b869
WiP: pathways module and module section
Dec 9, 2024
8bee4ef
Merge remote-tracking branch 'origin/main' into pathways-base
mattb-hippo Dec 9, 2024
e7af8bc
WiP: initial stab at Pathways Contents Page
mattb-hippo Dec 9, 2024
d80db41
WiP: pathways module and section scripts
Dec 10, 2024
df3d81f
WiP: added section start nav links
mattb-hippo Dec 10, 2024
b228dc9
spelling corrected on unknown content type warning
Dec 10, 2024
8387947
formatting error in ContentController.ServerTests.cs fixed
Dec 10, 2024
dace94b
migrations file updated for pathways module and section
Dec 10, 2024
5ace689
WiP: add warnings about incomplete module sections
mattb-hippo Dec 11, 2024
c89484d
Merge branch 'pathways-base' of github.com:DFE-Digital/childrens-soci…
mattb-hippo Dec 11, 2024
f12c40d
Merge branch 'next' into pathways-base
mattb-hippo Dec 11, 2024
29d58c2
Merge branch 'pathways-base' of github.com:DFE-Digital/childrens-soci…
mattb-hippo Dec 11, 2024
5d2f4be
First tests for Pathways Navigation Helper
mattb-hippo Dec 11, 2024
192c773
Merge branch 'pathways-base' into SFSW-2659-Create-Contents-Page-rend…
mattb-hippo Dec 11, 2024
c7a28f5
filled out tests for PathwaysOverviewPage
mattb-hippo Dec 11, 2024
d40a85c
commented out stubs not implemented yet
mattb-hippo Dec 11, 2024
e65d625
Merge branch 'pathways-base' into SFSW-2659-Create-Contents-Page-rend…
mattb-hippo Dec 11, 2024
d854d51
feat: Pathways Contents Page
mattb-hippo Dec 12, 2024
a8ceff9
made field readonly
mattb-hippo Dec 12, 2024
cc60a09
applying sonarcloud rule suggestions
mattb-hippo Dec 12, 2024
2e62246
applying sonarcloud rule suggestions
mattb-hippo Dec 12, 2024
8d64fd7
applying sonarcloud rule suggestions
mattb-hippo Dec 12, 2024
43a78ab
feat: pathways contents page rendering
mattb-hippo Dec 12, 2024
842b1f0
refactor to not use async
mattb-hippo Dec 12, 2024
a4ff5d0
refactor to reduce complexity, and use Count instead of Count()
mattb-hippo Dec 12, 2024
272b68d
removed button role per sonarqube
mattb-hippo Dec 12, 2024
25a01f2
hardening and unit tests
mattb-hippo Dec 12, 2024
1f4842f
final unit test for training content page
mattb-hippo Dec 13, 2024
d6c4610
Pathways Overview page rendering
mattb-hippo Dec 13, 2024
8646b92
amended pathways-module file
Dec 13, 2024
53ab09b
Merge branch 'pathways-base' of https://github.com/DFE-Digital/childr…
Dec 13, 2024
2dba8b2
Merge branch 'SFSW-2659-Create-Contents-Page-rendering' into pathways…
mattb-hippo Dec 13, 2024
6b989cb
Test SONAR_TOKEN_AS (#581)
as2024ad Dec 13, 2024
c2abab6
WiP: prev/next buttons
Dec 13, 2024
5f4d748
Completed navigation buttons
Dec 16, 2024
e1a4be0
Add location within training module into page banner
mattb-hippo Dec 16, 2024
cd99678
SonarCloud triggers.
as2024ad Dec 16, 2024
3a05608
button text replaced with variables
Dec 16, 2024
c741ebd
Merge pull request #587 from DFE-Digital/SFSW-2660-pathways-content-page
mattb-hippo Dec 16, 2024
6855951
Wording changed for ToC in .\0016-pathways-module.cjs
Dec 16, 2024
3c73bc4
Merge branch 'pathways-base' of https://github.com/DFE-Digital/childr…
Dec 16, 2024
e44a149
add feature flag and top-level nav for Pathways Training
mattb-hippo Dec 16, 2024
e5de7a9
Merge branch 'pathways-base' of github.com:DFE-Digital/childrens-soci…
mattb-hippo Dec 16, 2024
cc74cd8
added credit block as optional setting on training content pages
mattb-hippo Dec 17, 2024
95938f9
added breadcrumb alternating for pathways training content pages
mattb-hippo Dec 17, 2024
7e91ae4
extended column layout to handle four columns instead of just two or …
mattb-hippo Dec 17, 2024
c8f5e57
Merge branch 'next' into pathways-base
mattb-hippo Dec 17, 2024
325633a
Test trigger. (#591)
as2024ad Dec 17, 2024
714ef20
build(deps): bump docker/login-action from 1.9.0 to 3.3.0 (#576)
dependabot[bot] Dec 17, 2024
6f39cca
build(deps): bump actions/upload-artifact from 3 to 4 (#577)
dependabot[bot] Dec 17, 2024
bad82ab
build(deps): bump docker/metadata-action from 3.3.0 to 5.6.1 (#579)
dependabot[bot] Dec 17, 2024
8ff55a7
build(deps): bump docker/setup-buildx-action (#585)
dependabot[bot] Dec 17, 2024
a983096
new page type: All pathways overview page
Dec 17, 2024
9b4a2b6
Merge branch 'next' into pathways-base
Maria-C1 Dec 17, 2024
2d66457
build(deps): bump hashicorp/setup-terraform from 2 to 3 (#578)
dependabot[bot] Dec 17, 2024
d15b1f7
Merge branch 'next' into pathways-base
mattb-hippo Dec 17, 2024
6f5c1fc
unnecessary button script removed
Dec 17, 2024
1562d72
updated archive with lastest changes to new scripts
mattb-hippo Dec 17, 2024
bd397a9
replace nested ternary operators with more legible switch
mattb-hippo Dec 17, 2024
af50c3f
tidying whitespace
mattb-hippo Dec 18, 2024
b1f820e
refactored for better style
mattb-hippo Dec 18, 2024
1a79079
Agency Resources Url in top nav banner amended
Dec 18, 2024
af4d46a
Revert "Agency Resources Url in top nav banner amended"
Dec 18, 2024
b1c7ffb
Pathways Url amended in _Header and _ErrorLayout
Dec 18, 2024
797192e
moved section location info into separate partial
mattb-hippo Dec 18, 2024
ac1e995
Merge branch 'pathways-base' of github.com:DFE-Digital/childrens-soci…
mattb-hippo Dec 18, 2024
2cda048
Merge pull request #583 from DFE-Digital/pathways-base
Maria-C1 Dec 18, 2024
346e8c5
made pathways navigation cope with contents page not being specified
mattb-hippo Dec 19, 2024
21beb33
Merge pull request #597 from DFE-Digital/SFSW-2714-Support-pathways-m…
Maria-C1 Dec 19, 2024
c9b655e
fix: refactor to remove ternary operators
mattb-hippo Dec 19, 2024
55ef984
Merge branch 'next' into SFSW-2714-Support-pathways-modules-without-T…
mattb-hippo Dec 19, 2024
984db46
Merge pull request #598 from DFE-Digital/SFSW-2714-Support-pathways-m…
mattb-hippo Dec 19, 2024
4facecd
build(deps): bump azure/webapps-deploy from 2 to 3 (#596)
dependabot[bot] Jan 7, 2025
d81be11
build(deps): bump azure/login from 1 to 2 (#595)
dependabot[bot] Jan 7, 2025
f51e717
WiP: removing anti-forgery token, and disabling search service
mattb-hippo Jan 9, 2025
4d47960
fix: update page types for Pathways
mattb-hippo Jan 9, 2025
0dda9f2
Merge pull request #601 from DFE-Digital/pathways-page-types-change
mattb-hippo Jan 9, 2025
d32b34c
fix: rebuilt migrations archive
mattb-hippo Jan 9, 2025
52cf5eb
Merge pull request #602 from DFE-Digital/fix-pathways-migrations
mattb-hippo Jan 9, 2025
a4da280
Merge branch 'next' into SFSW-2724-Search-Service-cookie-causes-site-…
mattb-hippo Jan 10, 2025
8537263
Revert "WiP: removing anti-forgery token, and disabling search service"
mattb-hippo Jan 10, 2025
bee7d67
removed anti forgery handling from feedback form
mattb-hippo Jan 10, 2025
98c7d21
Merge pull request #604 from DFE-Digital/SFSW-2724-Search-Service-coo…
mattb-hippo Jan 10, 2025
dc75089
Made training content page more robust when partially configured
mattb-hippo Jan 10, 2025
d066ff0
adjusted warning message
mattb-hippo Jan 10, 2025
b547539
Merge pull request #605 from DFE-Digital/SFSW-2743-detatched-training…
mattb-hippo Jan 10, 2025
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
10 changes: 10 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
version: 2
updates:

- package-ecosystem: github-actions
directory: /
schedule:
interval: 'daily'
labels:
- github-actions
- dependabot
12 changes: 6 additions & 6 deletions .github/workflows/app-deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,26 +31,26 @@ jobs:
run: docker pull ghcr.io/dfe-digital/childrens-social-care-cpd:${{inputs.tag}}

- name: Sign in to Azure
uses: azure/login@v1
uses: azure/login@v2
with:
creds: '{"clientId":"${{ secrets.TF_ARM_CLIENT_ID }}","clientSecret":"${{ secrets.TF_ARM_CLIENT_SECRET }}","subscriptionId":"${{ secrets.TF_ARM_SUBSCRIPTION_ID }}","tenantId":"${{ secrets.TF_ARM_TENANT_ID }}"}'

- name: Deploy Dev to Azure Web App
uses: azure/webapps-deploy@v2
uses: azure/webapps-deploy@v3
with:
app-name: 's185d01-chidrens-social-care-cpd-app-service'
images: 'ghcr.io/dfe-digital/childrens-social-care-cpd:${{ inputs.tag }}'
if: ${{ inputs.workspace == 'Dev' }}

- name: Deploy Test to Azure Web App
uses: azure/webapps-deploy@v2
uses: azure/webapps-deploy@v3
with:
app-name: 's185d02-chidrens-social-care-cpd-app-service'
images: 'ghcr.io/dfe-digital/childrens-social-care-cpd:${{ inputs.tag }}'
if: ${{ inputs.workspace == 'Test' }}

- name: Deploy Load-Test to Azure Web App
uses: azure/webapps-deploy@v2
uses: azure/webapps-deploy@v3
with:
app-name: 's185d03-chidrens-social-care-cpd-app-service'
slot-name: 'staging'
Expand All @@ -62,14 +62,14 @@ jobs:
if: ${{ inputs.workspace == 'Load-Test' }}

- name: Deploy Pre-Prod to Azure Web App
uses: azure/webapps-deploy@v2
uses: azure/webapps-deploy@v3
with:
app-name: 's185t01-chidrens-social-care-cpd-app-service'
images: 'ghcr.io/dfe-digital/childrens-social-care-cpd:${{ inputs.tag }}'
if: ${{ inputs.workspace == 'Pre-Prod' }}

- name: Deploy Prod to Azure Web App
uses: azure/webapps-deploy@v2
uses: azure/webapps-deploy@v3
with:
app-name: 's185p01-chidrens-social-care-cpd-app-service'
slot-name: 'staging'
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/auto-deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,12 @@ jobs:
steps:

- name: Sign in to Azure
uses: azure/login@v1
uses: azure/login@v2
with:
creds: '{"clientId":"${{ secrets.TF_ARM_CLIENT_ID }}","clientSecret":"${{ secrets.TF_ARM_CLIENT_SECRET }}","subscriptionId":"${{ secrets.TF_ARM_SUBSCRIPTION_ID }}","tenantId":"${{ secrets.TF_ARM_TENANT_ID }}"}'

- name: Automatically Deploy Dev to Azure Web App
uses: azure/webapps-deploy@v2
uses: azure/webapps-deploy@v3
with:
app-name: 's185d01-chidrens-social-care-cpd-app-service'
# slot-name: 'Dev'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/docker-content-regression-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ jobs:
- name: Run the content regression tests
run: cd browser-tests/content-regression-tests; docker-compose up --exit-code-from content-regression-tests

- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
name: Playwright report
if: always()
with:
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/docker-contentful-space-export.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,13 +46,13 @@ jobs:

# Workaround: https://github.com/docker/build-push-action/issues/461
- name: Setup Docker buildx
uses: docker/setup-buildx-action@79abd3f86f79a9d68a23c75a09a9a85889262adf
uses: docker/setup-buildx-action@6524bf65af31da8d45b59e8c27de4bd072b392f5

# Login against a Docker registry except on PR
# https://github.com/docker/login-action
- name: Log into registry ${{ env.REGISTRY }}
if: github.event_name != 'pull_request'
uses: docker/login-action@28218f9b04b4f3f62068d7b6ce6ca5b26e35336c
uses: docker/login-action@9780b0c442fbb1117ed29e0efdff1e18412f7567
with:
registry: ${{ env.REGISTRY }}
username: ${{ github.actor }}
Expand All @@ -62,7 +62,7 @@ jobs:
# https://github.com/docker/metadata-action
- name: Extract Docker metadata
id: meta
uses: docker/metadata-action@98669ae865ea3cffbcbaa878cf57c20bbf1c6c38
uses: docker/metadata-action@369eb591f429131d6889c46b94e711f089e6ca96
with:
images: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}

Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/docker-contentful-space-import.yml
Original file line number Diff line number Diff line change
Expand Up @@ -46,13 +46,13 @@ jobs:

# Workaround: https://github.com/docker/build-push-action/issues/461
- name: Setup Docker buildx
uses: docker/setup-buildx-action@79abd3f86f79a9d68a23c75a09a9a85889262adf
uses: docker/setup-buildx-action@6524bf65af31da8d45b59e8c27de4bd072b392f5

# Login against a Docker registry except on PR
# https://github.com/docker/login-action
- name: Log into registry ${{ env.REGISTRY }}
if: github.event_name != 'pull_request'
uses: docker/login-action@28218f9b04b4f3f62068d7b6ce6ca5b26e35336c
uses: docker/login-action@9780b0c442fbb1117ed29e0efdff1e18412f7567
with:
registry: ${{ env.REGISTRY }}
username: ${{ github.actor }}
Expand All @@ -62,7 +62,7 @@ jobs:
# https://github.com/docker/metadata-action
- name: Extract Docker metadata
id: meta
uses: docker/metadata-action@98669ae865ea3cffbcbaa878cf57c20bbf1c6c38
uses: docker/metadata-action@369eb591f429131d6889c46b94e711f089e6ca96
with:
images: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}

Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/docker-publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,13 +57,13 @@ jobs:

# Workaround: https://github.com/docker/build-push-action/issues/461
- name: Setup Docker buildx
uses: docker/setup-buildx-action@79abd3f86f79a9d68a23c75a09a9a85889262adf
uses: docker/setup-buildx-action@6524bf65af31da8d45b59e8c27de4bd072b392f5

# Login against a Docker registry except on PR
# https://github.com/docker/login-action
- name: Log into registry ${{ env.REGISTRY }}
if: github.event_name != 'pull_request'
uses: docker/login-action@28218f9b04b4f3f62068d7b6ce6ca5b26e35336c
uses: docker/login-action@9780b0c442fbb1117ed29e0efdff1e18412f7567
with:
registry: ${{ env.REGISTRY }}
username: ${{ github.actor }}
Expand All @@ -73,7 +73,7 @@ jobs:
# https://github.com/docker/metadata-action
- name: Extract Docker metadata
id: meta
uses: docker/metadata-action@98669ae865ea3cffbcbaa878cf57c20bbf1c6c38
uses: docker/metadata-action@369eb591f429131d6889c46b94e711f089e6ca96
with:
images: ${{ env.REGISTRY }}/${{ env.IMAGE_NAME }}

Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/load-testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:

# Install the latest version of Terraform CLI
- name: Setup Terraform
uses: hashicorp/setup-terraform@v2
uses: hashicorp/setup-terraform@v3

# Initialize a new or existing Terraform working directory by creating initial files, loading any remote state, downloading modules, etc.
- name: Terraform Init
Expand Down Expand Up @@ -76,7 +76,7 @@ jobs:
pip3 install charset_normalizer

- name: Sign in to Azure
uses: azure/login@v1
uses: azure/login@v2
with:
creds: '{"clientId":"${{ secrets.TF_ARM_CLIENT_ID }}","clientSecret":"${{ secrets.TF_ARM_CLIENT_SECRET }}","subscriptionId":"${{ secrets.TF_ARM_SUBSCRIPTION_ID }}","tenantId":"${{ secrets.TF_ARM_TENANT_ID }}"}'

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/run-content-regression-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ jobs:
- name: Run Playwright tests
run: npx playwright test

- uses: actions/upload-artifact@v3
- uses: actions/upload-artifact@v4
if: always()
with:
name: playwright-report
Expand Down
20 changes: 12 additions & 8 deletions .github/workflows/sonarqube.yml
Original file line number Diff line number Diff line change
@@ -1,9 +1,13 @@
name: Static code analysis

on:
workflow_dispatch:
# push:
# branches: ["main", "release/**", "feature/**", "fix/**"]
pull_request:
branches:
- main
- next
types: [opened, synchronize, reopened]
branches: ["next", "main", "release/**", "feature/**", "fix/**"]

jobs:
code-analysis:
name: Static code analysis
Expand Down Expand Up @@ -52,17 +56,17 @@ jobs:

- name: Build, Test and Analyze
env:
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
SONAR_TOKEN: ${{ secrets.SONAR_TOKEN_AS }}
run: |
dotnet-sonarscanner begin \
/k:"DFE-Digital_childrens-social-care-cpd" \
/o:"dfe-digital" \
/k:as2024ad_sfsw \
/o:as2024ad \
/d:sonar.qualitygate.wait=true \
/d:sonar.cs.vscoveragexml.reportsPaths=coverage.xml \
/d:sonar.exclusions="**/*.css,**/*.scss,**/Models/*,**/Program.cs,**/WebApplicationBuilderExtensions.cs,**/GraphQL/Queries/*,**/Contentful-Schema/migrations/*.cjs" \
/d:sonar.test.exclusions="Childrens-Social-Care-CPD-Tests/**/*,Terraform-bootstrap/Error-pages/*" \
/d:sonar.token="${{ secrets.SONAR_TOKEN }}" \
/d:sonar.token="${{ secrets.SONAR_TOKEN_AS }}" \
/d:sonar.host.url="https://sonarcloud.io"
dotnet build --no-incremental
dotnet-coverage collect --settings dotnet-cover-config.xml -f xml -o coverage.xml "dotnet test"
dotnet-sonarscanner end /d:sonar.token="${{ secrets.SONAR_TOKEN }}"
dotnet-sonarscanner end /d:sonar.token="${{ secrets.SONAR_TOKEN_AS }}"
2 changes: 1 addition & 1 deletion .github/workflows/terraform-destroy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ jobs:

# Install the latest version of Terraform CLI
- name: Setup Terraform
uses: hashicorp/setup-terraform@v2
uses: hashicorp/setup-terraform@v3

# Initialize a new or existing Terraform working directory by creating initial files, loading any remote state, downloading modules, etc.
- name: Terraform Init
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/terraform-fmt.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ jobs:

# Install the latest version of Terraform CLI
- name: Setup Terraform
uses: hashicorp/setup-terraform@v2
uses: hashicorp/setup-terraform@v3

# Checks that all Terraform configuration files adhere to a canonical format
- name: Terraform Format
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/terraform-plan.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ jobs:

# Install the latest version of Terraform CLI
- name: Setup Terraform
uses: hashicorp/setup-terraform@v2
uses: hashicorp/setup-terraform@v3

# Initialize a new or existing Terraform working directory by creating initial files, loading any remote state, downloading modules, etc.
- name: Terraform Init
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/terraform.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ jobs:

# Install the latest version of Terraform CLI
- name: Setup Terraform
uses: hashicorp/setup-terraform@v2
uses: hashicorp/setup-terraform@v3

# Initialize a new or existing Terraform working directory by creating initial files, loading any remote state, downloading modules, etc.
- name: Terraform Init
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,8 @@ public class EntityResolverTests
[TestCase("linkListCard", typeof(LinkListCard))]
[TestCase("pageContents", typeof(PageContents))]
[TestCase("pageContentsItem", typeof(PageContentsItem))]
[TestCase("pathwaysModule", typeof(PathwaysModule))]
[TestCase("pathwaysModuleSection", typeof(PathwaysModuleSection))]
[TestCase("pdfFileResource", typeof(PdfFileResource))]
[TestCase("quoteBox", typeof(QuoteBox))]
[TestCase("richTextBlock", typeof(RichTextBlock))]
Expand Down
Loading
Loading