datacite / akita

DataCite Commons
https://commons.datacite.org
MIT License
5 stars 3 forks source link

Fix ORCID claim issue #364

Closed bklaing2 closed 3 weeks ago

bklaing2 commented 3 weeks ago

Purpose

closes: https://github.com/datacite/datacite/issues/2110

Approach

The JWT token parsing was broken when switching the cookies from using react-cookie-consent to next/cookies. This fixes that, and additionally removes the duplicated apollo client in the Providers component

Open Questions and Pre-Merge TODOs

Learning

Types of changes

Reviewer, please remember our guidelines:

cypress[bot] commented 3 weeks ago

2 flaky tests on run #1241 ↗︎

0 55 0 0 Flakiness 2

Details:

Merge 02a9c303970e5c6fc679c1f43ae1e66a7760e2c1 into fc2a2fc29353c601d337a842497d...
Project: akita Commit: 9be42c056b ℹ️
Status: Passed Duration: 03:11 💡
Started: Jun 24, 2024 9:49 AM Ended: Jun 24, 2024 9:52 AM
Flakiness  statistics.test.ts • 1 flaky test • Tests View Output
Test Artifacts
Overview > header Test Replay Screenshots
Flakiness  personContainer.test.ts • 1 flaky test • Tests View Output
Test Artifacts
PersonContainer > facets Test Replay Screenshots

Review all test suite changes for PR #364 ↗︎

cypress[bot] commented 3 weeks ago

1 flaky test on run #1247 ↗︎

0 55 0 0 Flakiness 1

Details:

Fix ORCID claim issue (#364)
Project: akita Commit: d5861baf50
Status: Passed Duration: 03:58 💡
Started: Jun 25, 2024 1:53 PM Ended: Jun 25, 2024 1:57 PM
Flakiness  cypress/e2e/fundrefContainer.test.ts • 1 flaky test • Tests View Output
Test Artifacts
FundrefContainer > not in ror Test Replay Screenshots

Review all test suite changes for PR #364 ↗︎