Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

[$50] Logic for skills showing #36

Closed
maxceem opened this issue Dec 25, 2020 · 16 comments
Closed

[$50] Logic for skills showing #36

maxceem opened this issue Dec 25, 2020 · 16 comments

Comments

@maxceem
Copy link
Contributor

maxceem commented Dec 25, 2020

Initially, we had functionality for showing the required skills in popup by 2 columns with checkmarks and crosses. But as per #1 it was removed.

image

It feels that we actually need this to show skills for resources and for job candidates.

Resource Bookings

Currently, it looks like this. We show only the skills which user has, but we don't show which skills are required by the job here, and we don't show which of the required skills users has.

image

  • For Resource Bookings we have 2 kinds of skills:

    • Skills required by job
    • Skills which user (resource) has

    When we show popup I guess it would be good to show:

    • what skills are required by job
    • what of the skills required by job user has, and which user doesn't have
    • what other skills not required by job user has
  • So I think it would be good to show a popup with 2 columns and icons (checkmark, cross), though we can only rename columns:

    • First column says Required Job Skills which would list all the skills required by the job, and using a checkmark and cross indicate which of the skills user have
    • Second column says Other User Skills would list all the skills of the user which are not required by the job but user have them

Job Candidates

  • the same logic like for Resource Bookings

Open Positions (Jobs)

  • keep as it is with only one list of required skills

@wdprice what do you think?

@maxceem maxceem added the question Further information is requested label Dec 25, 2020
@maxceem maxceem added this to the v1.0 - Initial Launch milestone Dec 25, 2020
@wdprice
Copy link

wdprice commented Jan 4, 2021

Good idea @maxceem.

What is the LOE? I'm leaning towards pushing this to a future release.

@maxceem maxceem removed the question Further information is requested label Jan 4, 2021
@maxceem
Copy link
Contributor Author

maxceem commented Jan 4, 2021

I would target it the initial release as from API we have everything ready, and only have to update UI.

@maxceem maxceem changed the title Logic for skills showing [$50] Logic for skills showing Jan 4, 2021
@maxceem
Copy link
Contributor Author

maxceem commented Jan 4, 2021

Contest https://www.topcoder.com/challenges/30161161 has been created for this ticket.

This is an automated message for maxceem via Topcoder X

@maxceem
Copy link
Contributor Author

maxceem commented Jan 4, 2021

@yoution you may pick up this after Connect tasks, this can wait.

@yoution
Copy link
Contributor

yoution commented Jan 5, 2021

@maxceem please assign to me

@yoution yoution mentioned this issue Jan 5, 2021
@yoution
Copy link
Contributor

yoution commented Jan 5, 2021

@maxceem please review

@maxceem
Copy link
Contributor Author

maxceem commented Jan 5, 2021

Contest https://www.topcoder.com/challenges/30161161 has been updated - it has been assigned to yoution.

This is an automated message for maxceem via Topcoder X

@maxceem
Copy link
Contributor Author

maxceem commented Jan 6, 2021

This ticket was not processed for payment. If you would like to process it for payment, please reopen it, add the tcx_FixAccepted label, and then close it again

This is an automated message for maxceem via Topcoder X

@maxceem maxceem reopened this Jan 6, 2021
yoution added a commit to yoution/taas-app that referenced this issue Jan 6, 2021
yoution added a commit to yoution/taas-app that referenced this issue Jan 6, 2021
@maxceem
Copy link
Contributor Author

maxceem commented Jan 6, 2021

@SathyaJayabal
Copy link
Collaborator

@maxceem ,

  1. The skill match popup is displayed only when the number of skills are more and we have a more link. We should have the popup show up on mouse over the "% skill matched" text, so its available for all cases. This is because now the popup is not just a list of extra skills that you cant show on the page and has extra information.

Screenshot 2021-01-07 at 9 59 04 AM

  1. When there are many skills in the popup the page height should be increased to accommodate the bigger popup so that user can scroll and view all skills in the popup.
    example: more link on the below page
    https://platform.topcoder-dev.com/taas/myteams/16786/positions/c1f3e159-729e-4eae-a9db-4f445a37e968
skill.popup-scroll.mov

@maxceem
Copy link
Contributor Author

maxceem commented Jan 7, 2021

@SathyaJayabal to keep the current issue scope as it is, can we handle 2 new requirements as a separate issue? Regarding 1, we have such design form the beginning see marvel https://marvelapp.com/prototype/921gg0f/screen/73987508/handoff and it was only required to show it on more even though there were checkmarks. So what you are suggesting is good enhancement for this feature.

And regarding 2, I'm not sure how this should work to make it comfortable, but we can think about it.

@SathyaJayabal
Copy link
Collaborator

Sure @maxceem , please create a new issue in case you want to add 1. for future release.

@maxceem
Copy link
Contributor Author

maxceem commented Jan 7, 2021

Created a follow-up issue #45

@maxceem maxceem closed this as completed Jan 7, 2021
@maxceem
Copy link
Contributor Author

maxceem commented Jan 7, 2021

Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30161161

This is an automated message for maxceem via Topcoder X

@maxceem
Copy link
Contributor Author

maxceem commented Jan 8, 2021

Opening for testing Topcoder X behavior.

@maxceem maxceem reopened this Jan 8, 2021
@maxceem
Copy link
Contributor Author

maxceem commented Jan 8, 2021

Closing for testing Topcoder X behavior.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants