Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CON/TP/Log-in]: Re-implement cross-platform log-in #706

Closed
katamatata opened this issue Apr 24, 2023 · 0 comments · Fixed by #751, #752 or #750
Closed

[CON/TP/Log-in]: Re-implement cross-platform log-in #706

katamatata opened this issue Apr 24, 2023 · 0 comments · Fixed by #751, #752 or #750
Labels
Area/backend [loopback/nestjs] in the backend side application Area/frontend [react] in the client side application Bug report Something isn't working Job Fair Priority: High This ticket has a high priority Task

Comments

@katamatata
Copy link
Contributor

katamatata commented Apr 24, 2023

Context/background

After Salesforce integration, the cross-platform log-in feature doesn't work, and we want to re-implement it. Users with a TP jobseeker profile should be able to log in to CON with the same credentials (a mentee profile should be automatically created). And vice versa, users with a CON mentee profile should be able to log in to TP with the same credentials (a jobseeker profile should be automatically created).

What needs to be done?

The logic needs to go something like this:

  1. User tries to log into CON and the front-end tries to load the ConProfile. Or, user logs into TP and the front-end tries to load the TpJobseekerProfile.
  2. Back-end doesn't find ConProfile or TpJobseekerProfile
  3. In this case, back-end tries to...
    a. for ConProfile, check if user has an existing TpJobseekerProfile. If so create a new ConProfile based on the data available in TpJobseekerProfile (a mapping)
    b. for TpJobseekerProfile ... vice versa
  4. Return the newly created profile record

The appropriate place to insert this logic in the backend is the code that handles the first query that the CON or TP front-end tries to run to load the desired profile record.
Furthermore, refer to the old loopback cross-platform log-in code that handled this specific case. Look here for it: https://github.com/talent-connect/connect/blob/282df3b1d474efa1f4ada0f458b55f094073ed24/apps/api/common/models/red-user.js

@katamatata katamatata added Area/frontend [react] in the client side application Area/backend [loopback/nestjs] in the backend side application Task labels Apr 24, 2023
@katamatata katamatata added the Bug report Something isn't working label Apr 24, 2023
@fortini fortini added Priority: High This ticket has a high priority Job Fair labels May 15, 2023
@fortini fortini added Priority: Medium This ticket has a medium priority and removed Priority: High This ticket has a high priority Job Fair labels May 23, 2023
@katamatata katamatata added Priority: High This ticket has a high priority Job Fair and removed Priority: Medium This ticket has a medium priority labels Jun 8, 2023
@ligiamxm ligiamxm moved this to Done & Deployed in TP: ReDI Talent Pool Jun 13, 2024
@ligiamxm ligiamxm moved this to Done & Deployed in CON: ReDI Connect Jun 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area/backend [loopback/nestjs] in the backend side application Area/frontend [react] in the client side application Bug report Something isn't working Job Fair Priority: High This ticket has a high priority Task
Projects
None yet
2 participants