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

OCD-4705: Fixes related to uploading and confirming via file #1736

Open
wants to merge 12 commits into
base: staging
Choose a base branch
from

Conversation

kekey1
Copy link
Contributor

@kekey1 kekey1 commented Nov 5, 2024

No description provided.

@kekey1 kekey1 requested a review from tmy1313 as a code owner November 5, 2024 18:15
andlar
andlar previously approved these changes Nov 5, 2024
tmy1313
tmy1313 previously approved these changes Nov 6, 2024
@kekey1 kekey1 dismissed stale reviews from andlar and tmy1313 via 5819961 November 13, 2024 15:33
andlar
andlar previously approved these changes Nov 13, 2024
This was deleted (by accident??) in OCD-4660 and we really need it! A
whole host of issues appear when it is not present.

[#OCD-4705]
andlar
andlar previously approved these changes Nov 14, 2024
When reviewing the checklist of things that should happen when a9 gets
removed, I realized that certain criteria are considered "editable" -
that is if they have been removed less than 1 year ago. I was including
ALL criteria in the listing CSV File but I think that might be confusing
for users actually because they would see some "original" criteria that
actually could not be added anyway. This commit changes the criteria
included in the download file - only those that are 1) attested or 2)
editable to the user are included. If ADMIN/ONC gets the CSV file then
they do get all the original criteria because they can do pretty much
whatever they want.

[#OCD-4705]
@kekey1 kekey1 changed the title OCD-4705: Merge participant ids independently of task ids OCD-4705: Fixes related to uploading and confirming via file Nov 18, 2024
Certification Status reviewer must be run regardless of the (updated)
status of the listing. If the "Active" status is removed, the updated
listing is no longer Active but validation should still be run as that
may have been the only status.

[#OCD-4705]
andlar
andlar previously approved these changes Nov 18, 2024
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.

3 participants