Progressive loading for schema validator output

[nellh]

This avoids loading the BIDS validator output for schema validator datasets, instead loading this once the validation drop down is clicked. For many datasets this avoids several megabytes of data transfer and potentially expensive API requests until this is actually needed.

A related bug fix is included to more accurately disable creating snapshots when the schema validator output contains errors or the draft has no author values provided.

See #3313

[codecov]

Codecov Report

Attention: Patch coverage is 53.30189% with 99 lines in your changes missing coverage. Please review.

Project coverage is 47.21%. Comparing base (769d876) to head (c1325b2).
Report is 10 commits behind head on master.

Files with missing lines	Patch %	Lines
...src/scripts/validation/validation-results-query.ts	56.81%	19 Missing ⚠️
...enneuro-server/src/graphql/resolvers/validation.ts	0.00%	19 Missing ⚠️
...-app/src/scripts/validation/validation-results.tsx	48.38%	16 Missing ⚠️
...nneuro-app/src/scripts/dataset/routes/snapshot.tsx	0.00%	13 Missing ⚠️
...penneuro-app/src/scripts/validation/validation.tsx	79.31%	12 Missing ⚠️
...ro-components/src/search-page/SearchResultItem.tsx	26.66%	11 Missing ⚠️
packages/openneuro-server/src/graphql/schema.ts	0.00%	5 Missing ⚠️
...enneuro-app/src/scripts/uploader/upload-issues.tsx	0.00%	2 Missing ⚠️
...nneuro-app/src/scripts/dataset/draft-container.tsx	0.00%	1 Missing ⚠️
...pp/src/scripts/dataset/routes/tab-routes-draft.tsx	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #3322      +/-   ##
==========================================
- Coverage   47.72%   47.21%   -0.51%     
==========================================
  Files         590      591       +1     
  Lines       37666    37768     +102     
  Branches     1272     1243      -29     
==========================================
- Hits        17976    17834     -142     
- Misses      19503    19747     +244     
  Partials      187      187              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[thinknoack]

reviewing the app locally looks good. I see loading animations for both draft and snapshots. Code looks good, tests pass, and no lint issues.

[sentry]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ TypeError: d is null /datasets/ds004828 View Issue

_{Did you find this useful? React with a 👍 or 👎}