Skip to content

fix(GAT-9247): Fixes GSM failure while calling old service no longer …#1707

Merged
loki-sinclair-hdruk merged 1 commit into
devfrom
bug/GAT-9247
Jul 21, 2026
Merged

fix(GAT-9247): Fixes GSM failure while calling old service no longer …#1707
loki-sinclair-hdruk merged 1 commit into
devfrom
bug/GAT-9247

Conversation

@loki-sinclair-hdruk

@loki-sinclair-hdruk loki-sinclair-hdruk commented Jul 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

…available

Screenshots (if relevant)

Describe your changes

Issue ticket link

Environment / Configuration changes (if applicable)

Requires migrations being run?

If not using the pre-push hook. Confirm tests pass:

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added appropriate unit tests
  • I have created mocks for unit tests (where appropriate)
  • I have added appropriate Behat tests to confirm AC (if applicable)
  • I have added Swagger annotations for new endpoints (if applicable)
  • I have added audit logs for new operation logic (if applicable)
  • I have added new environment variables to the .env.example file (if applicable)
  • I have added new environment variables to terraform repository (if applicable)

@loki-sinclair-hdruk
loki-sinclair-hdruk requested review from a team as code owners July 21, 2026 13:06
@gh-actions-pipelines-app

gh-actions-pipelines-app Bot commented Jul 21, 2026

Copy link
Copy Markdown

🚨 Invalid PR title format!

Your PR title must follow one of these formats:

  • feat(GAT-1234): Your title
  • fix!(GAT-5678): Breaking change
  • RELEASE: vX.Y.Z

Please update your PR title accordingly. 😊

3 similar comments
@gh-actions-pipelines-app

gh-actions-pipelines-app Bot commented Jul 21, 2026

Copy link
Copy Markdown

🚨 Invalid PR title format!

Your PR title must follow one of these formats:

  • feat(GAT-1234): Your title
  • fix!(GAT-5678): Breaking change
  • RELEASE: vX.Y.Z

Please update your PR title accordingly. 😊

@gh-actions-pipelines-app

gh-actions-pipelines-app Bot commented Jul 21, 2026

Copy link
Copy Markdown

🚨 Invalid PR title format!

Your PR title must follow one of these formats:

  • feat(GAT-1234): Your title
  • fix!(GAT-5678): Breaking change
  • RELEASE: vX.Y.Z

Please update your PR title accordingly. 😊

@gh-actions-pipelines-app

gh-actions-pipelines-app Bot commented Jul 21, 2026

Copy link
Copy Markdown

🚨 Invalid PR title format!

Your PR title must follow one of these formats:

  • feat(GAT-1234): Your title
  • fix!(GAT-5678): Breaking change
  • RELEASE: vX.Y.Z

Please update your PR title accordingly. 😊

@loki-sinclair-hdruk loki-sinclair-hdruk changed the title bug(GAT-9247): Fixes GSM failure while calling old service no longer … fix(GAT-9247): Fixes GSM failure while calling old service no longer … Jul 21, 2026
@gh-actions-pipelines-app

gh-actions-pipelines-app Bot commented Jul 21, 2026

Copy link
Copy Markdown

🎉 Great job! Your PR title follows the correct format. 🚀

@loki-sinclair-hdruk
loki-sinclair-hdruk merged commit 7d9e732 into dev Jul 21, 2026
2 of 6 checks passed
@loki-sinclair-hdruk
loki-sinclair-hdruk deleted the bug/GAT-9247 branch July 21, 2026 14:07
gh-actions-pipelines-app Bot pushed a commit that referenced this pull request Jul 24, 2026
@semantic-release-bot
chore(release): 2.46.0 [skip ci]
5b5bf4c 
## [2.46.0](v2.45.0...v2.46.0) (2026-07-24)

### ✨ Features

* **GAT-9018:** add queryable gwdm_version column to dataset_versions (#1698) ([e4d5d97](e4d5d97)), closes [GAT-9018](undefinedGAT-9018)
* **GAT-9075:** Typesense service and mocks initial commit (#1688) ([a776889](a776889)), closes [GAT-9075](undefinedGAT-9075)
* **GAT-9076:** Feature flag for Typesense (#1689) ([f46dbc7](f46dbc7)), closes [GAT-9076](undefinedGAT-9076)
* **GAT-9078:** groundwork changes for Typesense usage (#1690) ([3a45a51](3a45a51)), closes [GAT-9078](undefinedGAT-9078)
* **GAT-9079:** Begins to flesh out collection schema for searchable models (#1693) ([77680ed](77680ed)), closes [GAT-9079](undefinedGAT-9079)
* **GAT-9149:** Prevents AppCheck returning a NoSQL vulnerability inc… (#1694) ([27135b2](27135b2)), closes [GAT-9149](undefinedGAT-9149) [NoSQL](undefinedoSQL)
* **GAT-9212:** GWDM version context + handler hierarchy + DatasetService Rearrange (#1699) ([b0874f9](b0874f9)), closes [GAT-9212](undefinedGAT-9212)
* **GAT-9230:** record error and disable after final job failure ([879309d](879309d)), closes [GAT-9230](undefinedGAT-9230) [those](undefinedose)

### 🐛 Bug Fixes

* **GAT-0000:** Align CRUK dummy dataset fixtures with GWDM 2.1 schema. (#1686) ([9cf09e8](9cf09e8)), closes [GAT-0000](undefinedGAT-0000)
* **GAT-8925:** Fix incorrect config read on cohort expiry command (#1692) ([ecedbfc](ecedbfc))
* **GAT-9140:** added condition to take cloud-db backup before the release (#1696) ([9a67566](9a67566)), closes [GAT-9140](undefinedGAT-9140)
* **GAT-9222:** dataset-version linkage cleanup (#1704) ([248c5de](248c5de)), closes [GAT-9222](undefinedGAT-9222)
* **GAT-9247:** Correctly parse auth_secret_key to return when listing integration in FE (#1708) ([6c3df93](6c3df93)), closes [GAT-9247](undefinedGAT-9247)
* **GAT-9247:** Fixes GSM failure while calling old service no longer … (#1707) ([7d9e732](7d9e732))
* **GAT-9247:** properly form the path to avoid local credentials (#1709) ([a0ab378](a0ab378)), closes [GAT-9247](undefinedGAT-9247)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@equinoxmatt equinoxmatt equinoxmatt approved these changes
@Vikas00230 Vikas00230 Vikas00230 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@loki-sinclair-hdruk @equinoxmatt @Vikas00230