Bug: Admins/Supervisors Learning Hours Tab - Clicking on Volunteer name not routing to Volunteer's Learning Hours Page about casa HOT 5 CLOSED

Thank you for the clarification. Will look into this after completing other tickets. Thanks

from casa.

While evaluating implementation for current code to add test for issue #5513 I found that in currently implemented code there is issue in implementation as on click of volunteer name in supervisor learning page it is passing learning_hour.user_id and redirecting it to learning_hours controller show page which is trying to find learning hour using user id.

As per requirement mentioned in issue #5474 , we need to show all learning hours of Volunteer and not specific learning hour as implemented.

I also wanted to check instead of redirecting to Volunteer's learning hour page on click of Volunteer name, should we show for Supervisor and Casa Admin learning hours for Volunteer on its learning hour page itself as we do for Other duties so that view is similar across all pages.

from casa.

While evaluating implementation for current code to add test for issue #5513 I found that in currently implemented code there is issue in implementation as on click of volunteer name in supervisor learning page it is passing learning_hour.user_id and redirecting it to learning_hours controller show page which is trying to find learning hour using user id.

As per requirement mentioned in issue #5474 , we need to show all learning hours of Volunteer and not specific learning hour as implemented.

I also wanted to check instead of redirecting to Volunteer's learning hour page on click of Volunteer name, should we show for Supervisor and Casa Admin learning hours for Volunteer on its learning hour page itself as we do for Other duties so that view is similar across all pages.

Hi @sarvaiyanidhi thank you for bringing this to our attention. Yes, you are correct - we want the supervisors/admins to be taken to what the volunteers see on their learning hours page. Those details are important to the supervisors/admins. For Other duties - we can do the same where we take the supervisor/admin to what the volunteer sees. But if you like we can handle it on a separate ticket. Let me know what you prefer.

from casa.

Hi there, I am working on #5550 which is related. Just to let you know that my PR should also resolve this issue.

from casa.

Thank you Jade for the heads up.

from casa.