Update workflow actions and package references to latest versions#125
Merged
Conversation
Contributor
Dependency ReviewThe following issues were found:
License Issuessrc/XtremeIdiots.Portal.Integrations.Forums/XtremeIdiots.Portal.Integrations.Forums.csproj
src/XtremeIdiots.Portal.Web/XtremeIdiots.Portal.Web.csproj
OpenSSF ScorecardScorecard details
Scanned Files
|
There was a problem hiding this comment.
Pull request overview
Updates the Portal web app and forums integration to use the newer MX Invision Community API client packages and adjusts code to the new Result.Data response shape, while also pinning GitHub Actions references to versioned tags for more reproducible CI/CD.
Changes:
- Replace
XtremeIdiots.InvisionCommunitywithMX.InvisionCommunity.Api.*packages and update DI registration to the new builder-style configuration. - Refactor forums-client call sites to read data from
response.Result.Data. - Pin custom GitHub Actions references from
@mainto versioned tags across build/test/deploy/destroy workflows.
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/XtremeIdiots.Portal.Web/XtremeIdiots.Portal.Web.csproj | Bumps dependency versions (EF tools, MX/Portal API clients, GeoLocation client). |
| src/XtremeIdiots.Portal.Web/Startup.cs | Switches Invision client registration to the new builder pattern and namespace. |
| src/XtremeIdiots.Portal.Web/Auth/XtremeIdiots/XtremeIdiotsAuth.cs | Adapts member lookup and DTO typing to the new response/DTO model. |
| src/XtremeIdiots.Portal.Web/ApiControllers/HealthCheckController.cs | Updates forums health check to read CommunityUrl from Result.Data. |
| src/XtremeIdiots.Portal.Integrations.Forums/XtremeIdiots.Portal.Integrations.Forums.csproj | Replaces old InvisionCommunity package with MX.InvisionCommunity.Api.Client. |
| src/XtremeIdiots.Portal.Integrations.Forums/DemoManager.cs | Adapts downloads call to read Result.Data. |
| src/XtremeIdiots.Portal.Integrations.Forums/AdminActionTopics.cs | Adapts topic creation response handling to Result.Data. |
| .github/workflows/pr-verify.yml | Pins custom actions to versioned tags for PR verification, terraform plan/apply, and deploy. |
| .github/workflows/destroy-environment.yml | Pins terraform destroy action to a versioned tag. |
| .github/workflows/deploy-prd.yml | Pins CI/terraform/deploy actions to versioned tags for production pipeline. |
| .github/workflows/deploy-dev.yml | Pins CI/terraform/deploy actions to versioned tags for dev pipeline. |
| .github/workflows/build-and-test.yml | Pins CI and terraform plan actions to versioned tags. |
frasermolyneux
temporarily deployed
to
Development
GitHub Actions
Inactive
frasermolyneux
temporarily deployed
to
Development
GitHub Actions
Inactive
Contributor
⛔ Superseded — A newer run has replaced this result.⛔ Superseded — A newer run has replaced this result.🏗️ Terraform Plan
✅ Validate — Passed ✅ Plan
|
frasermolyneux
temporarily deployed
to
Development
GitHub Actions
Inactive
frasermolyneux
temporarily deployed
to
Development
GitHub Actions
Inactive
Contributor
⛔ Superseded — A newer run has replaced this result.🏗️ Terraform Plan
✅ Validate — Passed ✅ Plan
|
…dd cancellation token support
frasermolyneux
temporarily deployed
to
Development
GitHub Actions
Inactive
frasermolyneux
temporarily deployed
to
Development
GitHub Actions
Inactive
Contributor
🏗️ Terraform Plan
✅ Validate — Passed ✅ Plan
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request focuses on updating dependencies and refactoring code to support a new version of the Invision Community API client. The main changes include switching from the old
XtremeIdiots.InvisionCommunitypackage to the newMX.InvisionCommunity.Api.ClientandMX.InvisionCommunity.Api.Abstractionspackages, updating related code to handle new response structures, and updating several GitHub Actions to use specific versioned tags for reliability.Dependency and API Client Updates:
Replaced
XtremeIdiots.InvisionCommunitywithMX.InvisionCommunity.Api.ClientandMX.InvisionCommunity.Api.Abstractionsin both the integrations and web projects, updating all relevantusingstatements and package references. Also updated other related dependencies to newer versions. [1] [2] [3] [4] [5] [6] [7]Refactored service registration in
Startup.csto use the new API client’s builder pattern for configuration.Code Refactoring for New API Responses:
Result.Datainstead of direct properties. This affects topic creation, downloads, health checks, and user authentication flows. [1] [2] [3] [4] [5] [6]CI/CD Pipeline Improvements:
@dotnet-web-ci/v1.4instead of@main) for improved stability and reproducibility. This applies to build, test, deploy, and destroy workflows. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16]These changes modernize the integration with the forums API, improve code clarity and error handling, and ensure more reliable CI/CD processes.