Skip to content
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.

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

Include only CSV and TSV resources in import status dashboard #4317

Merged
merged 6 commits into from
Oct 23, 2024

Conversation

beeyayjay
Copy link
Contributor

Fixes [issue#]
The Datastore Import Status page includes datastores with resources that can't be imported, e.g. PDF and zip files. These resources display with a status of "Waiting", which is inaccurate and confusing.

(19021)

Describe your changes

Filter out distributions with resources that can't be imported, and datastores with only unimportable distributions from the data used to create the Datastore Import Status page.

QA Steps

  • Add manual QA steps in checklist format for a reviewer to perform. Be as specific as possible, provide examples if appropriate.

Checklist before requesting review

If any of these are left unchecked, please provide an explanation

  • I have updated or added tests to cover my code
  • I have updated or added documentation

@beeyayjay beeyayjay force-pushed the 19021-improve-import-status-dash branch from b82fd4f to d035d0f Compare October 17, 2024 20:24
@beeyayjay beeyayjay force-pushed the 19021-improve-import-status-dash branch from 5f713d4 to 13ccedc Compare October 21, 2024 18:53
@beeyayjay beeyayjay force-pushed the 19021-improve-import-status-dash branch from 83d15d9 to 666a628 Compare October 22, 2024 13:51
@beeyayjay beeyayjay force-pushed the 19021-improve-import-status-dash branch from 2bfbfa4 to e64a936 Compare October 23, 2024 15:06
@janette janette merged commit e57d833 into 2.x Oct 23, 2024
11 checks passed
@janette janette deleted the 19021-improve-import-status-dash branch October 23, 2024 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants