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

Feature/connectv2 with dev #315

Open
wants to merge 305 commits into
base: dev
Choose a base branch
from
Open

Feature/connectv2 with dev #315

wants to merge 305 commits into from

Conversation

vikasrohit
Copy link

fyi @gondzo @sushilshinde @mtwomey

Let me know if we can do this merge. This is very big merge and might be causing UI issues with community site.

Parth Shah and others added 30 commits August 24, 2016 14:13
… we won't need to add each individual icon file to exports)
exported few of the newly added icons
Fixed a few icon names to be consistent
… tc-includes to avoid multiple loading of css rules
-- Fixed console error in closing feature selector popup, it was because className property was of type SvgAnimatedString instead of plain old string type for svg children.
-- Fixed hover state for XMarkIcon
…e based on user's initials.

-- Updated Avatar component to use react-avatar library to render avatar based on user's name or handle
…e based on user's initials.

-- Fixed usage of src attribute
suppermancool and others added 25 commits January 24, 2019 14:34
fix fail select when search by key #2759
fix key up/down bound issue
Country selection - select by keyboard #2540
… last of the name field

— Should be fixed
…d redesign - Controls new design

+ fix to format prices with comma
…egistrationScreen and remove + sign requirement in phone numbers in Formsy.PhoneInput
Ignore 'dist' folder during npm run lint
Wizard redesign - Checkboxes and radio buttons with optional description and price (without design update)
@maxceem
Copy link
Collaborator

maxceem commented May 26, 2019

@vikasrohit it looks like community-app doesn't use this repository, see https://github.com/topcoder-platform/community-app/blob/develop/package.json#L33-L117. Community up uses a new https://github.com/topcoder-platform/topcoder-react-ui-kit instead.

So it looks like only Connect app uses this repo. Is there any chance that any other project uses it?

If no we can safely replace dev and master branch with our feature/connectv2 branch and don't merge them.

Also, we may consider removing the components which we don't use in Connect app from react-components as there are many which look like outdated not in use anymore, like an image gallery, for example.

@maxceem
Copy link
Collaborator

maxceem commented May 26, 2019

@vikasrohit I've just noticed, that Community app uses tc-accounts which uses react-components. But tc-accounts luckily uses our Connect branch https://github.com/appirio-tech/accounts-app/blob/dev/package.json#L15

@vikasrohit
Copy link
Author

Yep, tc-accounts started using react-components 7-8 months back when changed the registration flow for Connect and it required some components to be shared between accounts-app and connect-app.
For uses of react-components, even if community-app is not using it, there still might be few other old community side apps (e.g. topcoder-app etc) which still might be serving few pages of the overall topcoder experience. So, we need to do consolidate that list of apps where it might have been used with help of some Architect from the community side. So, lets leave it for now as is. We will take a look at it later on.

@sushilshinde
Copy link

@ThomasKranitsas please have a look

@vikasrohit
Copy link
Author

@sushilshinde @ThomasKranitsas we created this PR to analyze the feasibility of merging and syncing the connect specific branch with standard dev/master. However, if we are not using this repo at all after introducing https://github.com/topcoder-platform/topcoder-react-utils, we can safely convert connect specific branch as dev/master and later on we can move components to the https://github.com/topcoder-platform/topcoder-react-utils when we want to have single place for all react components for topcoder.

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

Successfully merging this pull request may close these issues.