Skip to content

Resolve draft modified time from commit history instead of cache #3174

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.

Sign up for GitHub

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

Merged
merged 3 commits into from
Oct 24, 2024

Merge branch 'master' into 3173-fix-draft-time-caching

9897641
Select commit
Loading
Failed to load commit list.
Merged

Resolve draft modified time from commit history instead of cache #3174

Merge branch 'master' into 3173-fix-draft-time-caching
9897641
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
Codecov / codecov/patch succeeded Oct 24, 2024 in 1s

56.52% of diff hit (target 44.16%)

View this Pull Request on Codecov

56.52% of diff hit (target 44.16%)

Annotations

Check warning on line 91 in packages/openneuro-server/src/datalad/dataset.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/openneuro-server/src/datalad/dataset.ts#L91 

Added line #L91 was not covered by tests

Check warning on line 101 in packages/openneuro-server/src/datalad/dataset.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/openneuro-server/src/datalad/dataset.ts#L101 

Added line #L101 was not covered by tests

Check warning on line 286 in packages/openneuro-server/src/graphql/resolvers/dataset.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/openneuro-server/src/graphql/resolvers/dataset.ts#L280-L286 

Added lines #L280 - L286 were not covered by tests

Check warning on line 230 in packages/openneuro-server/src/graphql/resolvers/snapshots.ts

See this annotation in the file changed.

@codecov codecov / codecov/patch

packages/openneuro-server/src/graphql/resolvers/snapshots.ts#L230 

Added line #L230 was not covered by tests
View more details on Codecov