Skip to content

fix for issue #4383 #4501

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
Closed
wants to merge 24 commits into from
Closed

fix for issue #4383 #4501

wants to merge 24 commits into from

Conversation

srjr19
Copy link
Contributor

@srjr19 srjr19 commented Jun 9, 2020

No description provided.

luizrrodrigues and others added 24 commits May 27, 2020 13:39
@luizrrodrigues
<Registrants> update propTypes with new param in V5 API
dfeec86
@luizrrodrigues
Fix regex to challengeId (to support UUID)
b9c25b0
@luizrrodrigues
Update prizeSets.type from Challenge Prize to placement
069bf7c
@luizrrodrigues
Updated URL params :challengeId to allow legacyID and UUID
4bd1737
@luizrrodrigues
Added validation if receive challenge tags before get recommendedChal…
a018d7e 
…lenges
@luizrrodrigues
Update vars to match with V5 API
6acb8ef
@luizrrodrigues
Fix lint
28e4c44
@luizrrodrigues
Merge branch 'issue-4378' into issue-4408
d76a92f
@rashmi73
Merge pull request #14 from topcoder-platform/integration-v5-challeng…
a635b03 
…e-api

Integration v5 challenge api
@rashmi73
Update index.jsx
6a533fa
@luizrrodrigues
Added default colorStyle to all members (temp solution)
6f21672
@luizrrodrigues
Reactivated the proxy-get to support calls to resources_api
f75b3d4
@luizrrodrigues
Challenge listing - Added check if challenge.events is empty
16b5896
@sushilshinde
Merge pull request #4487 from rashmi73/issue_4482
0777387 
Issue 4482 fix
@luizrrodrigues
Challenge Details - Fix isRegistered function
e36c4c4
@luizrrodrigues
Merge branch 'integration-v5-challenge-api' into issue-4408
e6fdf24
@rashmi73
Merge pull request #15 from topcoder-platform/integration-v5-challeng…
cd78714 
…e-api

Integration v5 challenge api
@rashmi73
Update index.jsx
bc39fa6
@rashmi73
Update index.jsx
5160e0d
@sushilshinde
Merge pull request #4489 from rashmi73/issue_4482
94f7383 
Issue 4482 fix
@sushilshinde
fix: for #4384
4e28176 
#4384	simran	#4481	topcoder-platform/topcoder-react-lib#181
		"#4483
#4484"	
			topcoder-platform/topcoder-react-lib#183
@sushilshinde
Merge pull request #4431 from topcoder-platform/issue-4408
8e48f49 
Issue#4408 : Fix registrants and submissions tabs in challenge details
@sushilshinde
fix: #4480,#4408,#4381
9c7d6ce 
#4408		#4431	topcoder-platform/topcoder-react-lib#179	1000.19.12
#4381				
#4480
@srjr19
fix for issue #4383
5f9eb6c
@srjr19 srjr19 mentioned this pull request Jun 9, 2020
Closed
@sushilshinde sushilshinde force-pushed the integration-v5-challenge-api branch from 9c7d6ce to 6b60d2a Compare June 9, 2020 10:44
sushilshinde added a commit to topcoder-platform/topcoder-react-lib that referenced this pull request Jun 11, 2020
@sushilshinde
npm. bump up 1000.19.13 for #4384
4be3737 
topcoder-platform/community-app#4383		topcoder-platform/community-app#4501	#184	
			#186	1000.19.13
@sushilshinde
Copy link
Collaborator

sushilshinde commented Jun 11, 2020 •
edited
Loading

@simranb86 too many files, this included files that I reverted

Please add your changes to latest branch, please PR again.

@srjr19
Copy link
Contributor Author

srjr19 commented Jun 11, 2020

@simranb86 too many files, this included files that I reverted

Please add your changes to latest branch, please PR again.

@sushilshinde when I opened PR, there were only my changes, looks like base was updated again after I did PR. :(

@sushilshinde
Copy link
Collaborator

Yes, please cherry pick your changes into latest branch.

Sorry about that.

@srjr19 srjr19 closed this Jun 11, 2020
sushilshinde added a commit that referenced this pull request Jun 11, 2020
@sushilshinde
fix: for #4383
09e64ae 
#4383		#4501	topcoder-platform/topcoder-react-lib#184	
		#4514	topcoder-platform/topcoder-react-lib#186	1000.19.13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ajefts ajefts Awaiting requested review from ajefts ajefts is a code owner

@sushilshinde sushilshinde Awaiting requested review from sushilshinde sushilshinde is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@srjr19 @sushilshinde @luizrrodrigues @rashmi73