aria-at-app icon indicating copy to clipboard operation
aria-at-app copied to clipboard

Collection Job - Per-test status updates

Open gnarf opened this issue 1 year ago • 4 comments

Implementing the backend updates for #818

gnarf avatar Apr 04 '24 19:04 gnarf

I'd like to formalize the distinction between "new way" and "old way" a bit. Doing so will further clarify the differences, give us more confidence when removing the deprecated logic in the future, and reduce the number of unnecessary code paths in the interim.

We've landed the automation-harness patch so it should be safe to remove the deprecated paths entirely, would you prefer I do just that?

gnarf avatar Apr 25 '24 14:04 gnarf

I think that could work. Don't we also have to update the callback URLs?

jugglinmike avatar Apr 25 '24 20:04 jugglinmike

I realized while updating the PR to remove the back-compat URLs that some mock work I needed had already been done on the working UI branch so I merged that into this one instead of keeping them separate. I could tease the frontend updates out of this branch if people still prefer. The changes from d87280b adds the frontend for the test queue and manage nvda run dialogs and some ways to test other status.

gnarf avatar May 01 '24 15:05 gnarf

Alright - front end tests are now passing as well, so this should be all good - I removed the back-compat URLs as well because the automation harness is properly working with the new style URLs already.

gnarf avatar May 01 '24 18:05 gnarf