Skip to content

Member Profiles #846

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 387 commits into from
Jul 31, 2023
Merged

Member Profiles #846

merged 387 commits into from
Jul 31, 2023

Conversation

kkartunov
Copy link
Collaborator

@kkartunov kkartunov commented Jul 28, 2023 •
edited
Loading

What's in this PR?

Member Profiles App https://topcoder.atlassian.net/browse/MP-90
Profile Nudge App https://topcoder.atlassian.net/browse/MP-70
Onboarding App https://topcoder.atlassian.net/browse/MP-75
Account Settings https://topcoder.atlassian.net/browse/MP-2
Updates, Fixes and Deployment of all apps affected by the production release.

vas3a and others added 30 commits July 11, 2023 15:39
@vas3a
#716 - limit the max no of skills selected in onboarding
c4704cf
@vas3a
Merge pull request #737 from topcoder-platform/716_limit-max-skills
80c9ebb 
#716 - limit the max no of skills selected in onboarding
@jmgasper
Merge pull request #735 from topcoder-platform/onboarding-mobile-views
b3c83dc 
Onboarding - mobile views
@kkartunov
MP-119 sort order of emsi skills
9469b12
@kkartunov
MP-418 trash icon on Safari fix
b09a8c6
@kkartunov
Merge pull request #741 from topcoder-platform/profiles-app
f81a7a9 
Profiles app
@suppermancool
Onboarding - Use different field for bio
c728a4e
@suppermancool
Onboarding - Next on last screen
122ea41
@jmgasper
Merge pull request #739 from topcoder-platform/onboarding
39bd999 
Onboarding mobile views
@jmgasper
Merge pull request #742 from topcoder-platform/issue-738
471e36f 
Onboarding - Use different field for bio + Next on last screen
@suppermancool
- Onboarding - Profile pic popup
8e379c6 
- Icons missing in Safari
@jmgasper
Merge pull request #748 from topcoder-platform/issue-743
0f9f052
@vas3a
remove stripe tokens
7bc2a33
@vas3a
react-env: convert env variables to bool/number/null
fd080e1
@jmgasper
Merge pull request #754 from topcoder-platform/onboarding
d1835ea 
Onboarding
@testflyjets
Changes CircleCI env var handling
1d4baa7
@vas3a
Rely on "LOGICAL_ENV" for the build mode
9076b17
@suppermancool
onboarding issues fix
520eab1
@vas3a
remove onboarding skill limit
deea36b
@vas3a
Merge pull request #757 from topcoder-platform/onboarding-remove-limit
5b285f4 
remove onboarding skill limit -> dev
@jmgasper
Merge pull request #756 from topcoder-platform/onboarding-minor-fixes
e8e14cb
@vas3a
Merge pull request #755 from topcoder-platform/LVT-135-modify-circle-…
35d7690 
…env-vars

Changes CircleCI env var handling
@vas3a
Merge pull request #749 from topcoder-platform/env-fixes
7f981b4 
Env fixes
@suppermancool
Onboarding - Tab order
d15c710
@vas3a
log build mode
852fa4e
@vas3a
Merge pull request #758 from topcoder-platform/env-fixes
4762ca7 
log build mode
@jmgasper
Merge pull request #759 from topcoder-platform/onboarding-minor-fixes
4ea438a
@kkartunov
MP-176 job dates cond
514e47b
@kkartunov
MP-73 select how to show names
456ac0c
@kkartunov
Merge pull request #760 from topcoder-platform/profiles-app
565a685 
Profiles app -> dev
vas3a and others added 9 commits July 27, 2023 10:15
@vas3a
TAL-36 - update restricted routes to wait for profile data before ren…
1b5a6b3 
…dering
@vas3a
Merge pull request #843 from topcoder-platform/TAL-36_talent-search-b…
664c009 
…ehind-admin

TAL-36 - await restricted route user roles
@vas3a
MP-301 - hide the language section in profiles when no languages were…
107f99e 
… added
@vas3a
MP-302 - add placeholders for bio modal form
df69bc8
@vas3a
Merge pull request #844 from topcoder-platform/MP-301_hide-language-s…
937df53 
…ection-when-empty

MP-301 hide language section when empty -> dev
@vas3a
MP-303 - add am/pm for local timezone for profiles
67b2883
@vas3a
MP-303 - also show timezone
c63becd
@kkartunov
Merge pull request #845 from topcoder-platform/MP-303_timezone-show-ampm
0001bd6 
MP-303 - add am/pm for local timezone for profiles -> dev
@kkartunov
MP-295 develop modal init
e18a4ce
vas3a
vas3a approved these changes Jul 28, 2023
View reviewed changes
Copy link
Collaborator

@vas3a vas3a left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a pretty large merge, it looks good overall; I see nothing that stands out 👍

testflyjets
testflyjets approved these changes Jul 28, 2023
View reviewed changes
Copy link
Contributor

@testflyjets testflyjets left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are there any additional updates to config files in S3 that need to happen with this deployment? If so, please capture them in the Run Book so we don't forget. Nice work!

suppermancool and others added 9 commits July 29, 2023 12:52
@vas3a vas3a merged commit 166bda3 into master Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vas3a vas3a vas3a approved these changes

@jmgasper jmgasper Awaiting requested review from jmgasper

+1 more reviewer

@testflyjets testflyjets testflyjets approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@kkartunov @testflyjets @vas3a @jmgasper @suppermancool