-
Notifications
You must be signed in to change notification settings - Fork 24
[$60] ResourceBookings Edit UI: Should show related Job name #90
Comments
Challenge https://www.topcoder.com/challenges/88578d52-3659-404f-9fac-b58bec815c62 has been created for this ticket. |
@bug-bash-helper assign |
@dashu-baba ✅ you are now assigned to this issue and have 12 hours to complete it. As soon as you are done, please, make a comment like below, including the link to the pull request:
|
Challenge https://www.topcoder.com/challenges/88578d52-3659-404f-9fac-b58bec815c62 has been assigned to nowshad.bappy. |
@bug-bash-helper unassign |
@dashu-baba ✅ you have been unassigned from this issue. Now you may pick up another issue which is open for pickup if you like to. |
Challenge https://www.topcoder.com/challenges/88578d52-3659-404f-9fac-b58bec815c62 nowshad.bappy has been unassigned. |
@bug-bash-helper unassign |
@yoution 🛑 I cannot unassign you from this issue because you are not assigned. |
@bug-bash-helper assign |
@yoution ✅ you are now assigned to this issue and have 12 hours to complete it. As soon as you are done, please, make a comment like below, including the link to the pull request:
|
Challenge https://www.topcoder.com/challenges/88578d52-3659-404f-9fac-b58bec815c62 has been assigned to yoution. |
@maxceem Resource booking page both View and Edit are not displaying sometimes and getting displayed sometimes. It seems like a intermittent issue. Below are some of the issues I noted. After refresh ,the page is displayed Also the Job name was different.
|
@sandhiyakavi 1 & 2 are fixed by ee297cb. 3 was already fixed by 7d037b7 (note, that issue 3 has been reproducible on each time, it got broken during one of the previous fix). Regarding showing Ready for QA now. |
Verified on Dev Env. Working as expected.
bandicam.2021-02-19.19-17-00-770.mp4
bandicam.2021-02-19.19-18-08-554.mp4
bandicam.2021-02-19.19-19-05-279.mp4 |
Payment failed: [400]: Failed to activate challenge. Detail: Cannot Activate this project, it has no active billing accounts. |
Payment task has been updated: https://www.topcoder.com/challenges/88578d52-3659-404f-9fac-b58bec815c62 |
@sandhiyakavi as discussed pls add the latest screenshots of this screen from Dev - seems like it has changed ... |
@maxceem The title for the Resource booking view and Edit page has been changed from "Member Details" and "Edit Member Details" to "Resource Booking" and "Edit Resource Booking" in Dev as well as Prod. Is it expected? Because I was not able to find an issue logged related to this. |
yes, @sandhiyakavi it's expected, I've confirmed it with Will here #56 (comment) |
Follow up from #56 (comment)
This issue in some way happens to both pages:
view: https://platform.topcoder-dev.com/taas/myteams/16926/rb/a8b9fda7-df1c-4fe2-ba4b-b76142ece345
edit: https://platform.topcoder-dev.com/taas/myteams/16926/rb/a8b9fda7-df1c-4fe2-ba4b-b76142ece345/edit:
This happens because both these pages are implemented incorrectly.
Both these pages are trying to show information from the Candidate instead of the Job Name:
view source code: https://github.com/topcoder-platform/taas-app/blob/dev/src/routes/ResourceBookingDetails/index.jsx#L24
edit source code: https://github.com/topcoder-platform/taas-app/blob/dev/src/routes/ResourceBookingForm/index.jsx#L39-L41
We don't need to to get candidates data for these pages.
We only need to show the Job Name on both of these pages.
Update the logic there so we only are loading the Job using
jobId
which is defined in the resource. And there should not be any logic for Candidate.Note, that
jobId
might benull
in the resource, in such case instead of the Job Name show text<Not Assigned>
.When page is loading wait until job data is loaded before showing the page. So we would see the page already with Job Name. But keep in mind that maybe there is not Job, then the page should be shown without Job Name and don't wait Job loading.
Make the logic clear for all of this.
Provide a link for testing, when there is not Job assigned to the resource.
It is preferable to use hook
useAsync
instead ofuseData
for loading data on both of these pages. I also think that with hookuseAsync
it would be easier to implement the logic for waiting for Job loading together withgetReourceBookingById
. (This is not a mandatory requirement though).The text was updated successfully, but these errors were encountered: