issue 3474 fix

[rashmi73]

this PR fixes issue topcoder-platform/community-app#3474

[urwithat]

@rashmi73 & @sushilshinde - I believe this is wrong:
We need to construct the following url:
https://api....com/v3/tags/?filter=domain%3DSKILLS%26status%3DAPPROVED&limit=1000

The current code will construct the url as follows:
/tags/?filter=domain=SKILLS&status=APPROVED&limit=1000

Firstly under the filter param we should have domain & skill and limit would be second param.
Secondly I don't see any conversion to URL escape codes for domain & skill

Kindly fix as per the url required to be formed: /v3/tags/?filter=domain%3DSKILLS%26status%3DAPPROVED&limit=1000

[rashmi73]

@urwithat changes have been updated in PR. kindly verify if this is what expected? I guess it should be working now.

@sushilshinde

[urwithat]

@rashmi73 - Hope this will handle conversion to URL escape codes for domain & skill
domain%3DSKILLS%26status%3DAPPROVED

[sushilshinde]

@rashmi73 - Hope this will handle conversion to URL escape codes for domain & skill
domain%3DSKILLS%26status%3DAPPROVED

@rashmi73 please check

[sushilshinde]

@rashmi73 are you fixing this and this is P1, or I will open for pick up again.

[rashmi73]

@sushilshinde I am working and it should work

[rashmi73]

@sushilshinde it as expected now @urwithat.

Did not take screenshot but it forms url perfectly now. I tested it

[rashmi73]

@sushilshinde @urwithat screenshot

[rashmi73]

@sushilshinde @urwithat you can merge it now.

[rashmi73]

@sushilshinde @urwithat waiting here for feedback or either merging of code

[sushilshinde]

@rashmi73 @urwithat lint is failing

https://circleci.com/gh/topcoder-platform/topcoder-react-lib/617

[rashmi73]

@sushilshinde new PR for lint, #117