Skip to content

feat(GAT-7957): added Github app authentication#1561

Merged
Vikas00230 merged 8 commits intodevfrom
GAT-7957
Feb 27, 2026
Merged

feat(GAT-7957): added Github app authentication#1561
Vikas00230 merged 8 commits intodevfrom
GAT-7957

Conversation

@Vikas00230
Copy link
Copy Markdown
Contributor

@Vikas00230 Vikas00230 commented Feb 27, 2026

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)

@gh-actions-pipelines-app
Copy link
Copy Markdown

gh-actions-pipelines-app Bot commented Feb 27, 2026

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

@Vikas00230 Vikas00230 merged commit eac54b5 into dev Feb 27, 2026
5 checks passed
@Vikas00230 Vikas00230 deleted the GAT-7957 branch February 27, 2026 14:36
gh-actions-pipelines-app Bot pushed a commit that referenced this pull request Mar 19, 2026
@semantic-release-bot
chore(release): 2.37.0 [skip ci]
4cdad63 
## [2.37.0](v2.36.0...v2.37.0) (2026-03-19)

### ✨ Features

* **GAT-0000:** Optimising auth login from ~3s to < 1s (#1558) ([be36fa8](be36fa8)), closes [GAT-0000](GAT-0000)
* **GAT-0000:** Refactoring to enable partner context for API operation for CRUK and others, and optimisations when noticed (#1569) ([6d6120b](6d6120b)), closes [GAT-0000](GAT-0000)
* **GAT-5218:** publications has keywords (#1575) ([951c7aa](951c7aa)), closes [GAT-5218](GAT-5218)
* **GAT-5397:** Include linkage type for publication card (#1574) ([f9be66a](f9be66a)), closes [GAT-5397](GAT-5397)
* **GAT-6015:** publication date (#1543) ([f4f7e69](f4f7e69)), closes [GAT-6015](GAT-6015)
* **GAT-7340:** Add data custodian network information to Datasets (#1566) ([aba7bdb](aba7bdb)), closes [GAT-7340](GAT-7340)
* **GAT-7376:** Create ability to run now for Predefined Integration (#1582) ([c1175e8](c1175e8)), closes [GAT-7376](GAT-7376)
* **GAT-7580:** Update endpoints to enable addition of Data Custodian name (#1568) ([3ccd2c0](3ccd2c0)), closes [GAT-7580](GAT-7580)
* **GAT-7768:** filter OOO emails (#1560) ([3c610e6](3c610e6)), closes [GAT-7768](GAT-7768)
* **GAT-7957:** added Github app authentication (#1561) ([eac54b5](eac54b5)), closes [GAT-7957](GAT-7957)
* **GAT-8092:** Update endpoint for Dataset & BioSample landing pages to include summary.datasetAliases (#1570) ([3d1e64d](3d1e64d)), closes [GAT-8092](GAT-8092) [BioSample](oSample)
* **GAT-8123:** homepage statistics (#1538) ([0742bdc](0742bdc)), closes [GAT-8123](GAT-8123)
* **GAT-8233:** DAR answer add new field (#1553) ([f2061a5](f2061a5)), closes [GAT-8233](GAT-8233)
* **GAT-8242:** Add action bar to DAR (#1539) ([3b2166e](3b2166e)), closes [GAT-8242](GAT-8242)
* **GAT-8242:** Fix OA comments (#1546) ([6ce346a](6ce346a))
* **GAT-8246:** Data Custodian Networks now return team_ids to help with filters (#1565) ([d38535b](d38535b)), closes [GAT-8246](GAT-8246)
* **GAT-8329:** Extend Gateway Metadata Integration (#1583) ([2b9fad8](2b9fad8)), closes [GAT-8329](GAT-8329) [Gateway](Gateway)
* **GAT-8384:** added release pipeline (#1547) ([20e328b](20e328b)), closes [GAT-8384](GAT-8384)
* **GAT-8384:** fixed lint errors (#1549) ([d857f61](d857f61))
* **GAT-8384:** fixed the PR title file (#1552) ([cc03927](cc03927))
* **GAT-8450:** Update existing and add any new emails required (#1571) ([1781b42](1781b42)), closes [GAT-8450](GAT-8450)
* **GAT-8465:** rework of widget colours (#1584) ([db490d8](db490d8)), closes [GAT-8465](GAT-8465)
* **GAT-8534:** update export metadata (#1540) ([32bae25](32bae25)), closes [GAT-8534](GAT-8534)
* **GAT-8586:** admin update cohort discovery state in datasets (#1562) ([abee8da](abee8da)), closes [GAT-8586](GAT-8586)
* **GAT-8610:** Fix access checks for scrambled open-athens emails (#1567) ([1c58b9a](1c58b9a))
* **GAT-8635:** Update search service to include keywords for Publications (#1576) ([a82d2a7](a82d2a7)), closes [GAT-8635](GAT-8635)

### 🐛 Bug Fixes

* **GAT-0000:** export metadata Distribution release date (#1544) ([76410ed](76410ed)), closes [GAT-0000](GAT-0000)
* **GAT-0000:** fix for the collection observer (#1579) ([a3e6bbe](a3e6bbe))
* **GAT-0000:** fix send email cohort request) (#1573) ([5853068](5853068))
* **GAT-0000:** Fix user on auth being returned as array instead of ob… (#1559) ([72df3d4](72df3d4))
* **GAT-7418:** TVS SDE datasets all have the same PID in the database (#1563) ([a07783a](a07783a)), closes [GAT-7418](GAT-7418)
* **GAT-7951:** Applicant organisation details are not being pulled through to enquiry template (#1557) ([87c0e74](87c0e74)), closes [GAT-7951](GAT-7951)
* **GAT-8106:** Reduce page load time search collections networks (#1555) ([84c5cab](84c5cab)), closes [GAT-8106](GAT-8106)
* **GAT-8119:** update notifications (#1548) ([ec1456e](ec1456e)), closes [GAT-8119](GAT-8119)
* **GAT-8530:** Structural Metadata download (#1572) ([86c0450](86c0450)), closes [GAT-8530](GAT-8530)
* **GAT-8579:** NHS SDE approval is wrong (#1556) ([ca5e345](ca5e345)), closes [GAT-8579](GAT-8579)
* **GAT-8640:** swagger update tags (#1577) ([0361eda](0361eda)), closes [GAT-8640](GAT-8640)
* **GAT-8664:** loading Data uses (#1581) ([ff14f83](ff14f83)), closes [GAT-8664](GAT-8664)
* **GAT-9999:** Fixes (DP-492) issues surrounding auth sometimes failing (#1542) ([8cc7cbc](8cc7cbc))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dnhdruk dnhdruk dnhdruk approved these changes

@soringumeni1 soringumeni1 soringumeni1 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

@Vikas00230 @dnhdruk @soringumeni1