Marks Transfer 23-24 TI1.5 Updates

[aydevworks]

CTP-2811 Improve UX when push is taking place
CTP-2812 hide push information about "submission log"
CTP-2923 Minor improvements for marks transfer plugin

[aspark21]

adding @stuartlamour to the review. I think we should try and find some time to review together during TI planning.

Had my initial look at this, noted a couple of improvements this does deliver:

• clicking on the "transfer records exists" icon takes you to the activity level view
• progress bar once the task is running

but also a couple of quirks:

• we're not setting expectations about how long it's going to take or what's going on. after a user clicks, it just starts whirling & would likely be assumed to be broken
• if people click on push from course level view, it's very unlikely they will see the transfer failures which are seen in the activity level view.
• we have removed the ability to transfer records from the activity level view entirely

In terms of how we address this, some of the ideas that come to mind:

• make clear that a task is queued & it could take an hour to process
• send user an email when transfer is complete with a list of any errors on the "grade push" aspect (i.e. not for submission log) that they will have to address through other means - that should link back to the activity view and some documentation about how they can be addressed which Kerry has worked on I believe
• (if "the transfer" is a result of a push all at course level, then maybe this should come together as one summary email once all tasks are complete rather than separate ones for each activity / adhoc task)

[aydevworks]

This PR is replaced by #32