[$300] Update the challenge object after forum created in processor

[jmgasper]

After we've created the group / forum for a challenge, we need to update the challenge object to contain a reference back to the forum created:

So you should be able to POST/PUT/PATCH with the challenge object as laid out in this git issue:
topcoder-platform/challenge-api#290

More information on the format is here: topcoder-platform/challenge-api#290 (comment)

From Topcoder:

And you should assume that there can be multiple vanilla forums for whatever reason. So you should loop through all the challenge objects in the array, and if they’re type: challenge, and provider: vanilla, and the url is empty, create that forum.

[jmgasper]

Contest https://www.topcoder.com/challenges/30144037 has been created for this ticket.

This is an automated message for ghostar via Topcoder X

[jmgasper]

Contest https://www.topcoder.com/challenges/30144037 has been updated - it has been assigned to obog.

This is an automated message for ghostar via Topcoder X

[atelomycterus]

@jmgasper Not quite everything is clear. Example:

"discussions": [
     {
       "provider": "vanilla",
       "name": "Justin Forum Test 11 Discussion",
       "type": "challenge"
     },
     
      {
       "provider": "vanilla",
       "name": "Customer Discussion",
       "type": "customer"
     }
   ],

1. For a discussion with type='challenge':
   We create all vanilla entities (a group and so on.). Need to implement

'update the challenge object to contain a reference back to the forum created'

It's clear, no questions.

2. Challenge without 'discussions'. 'discussions' is not required in the challenge schema.
   Should we create a group only in Vanilla?
   

3. Now the main category of challenge's discussions is named as a challenge's name.
   Should we use 'name' from a discussion object?
   {
   "provider": "vanilla",
   "name": "Justin Forum Test 11 Discussion",
   "type": "challenge"
   }

And you should assume that there can be multiple vanilla forums for whatever reason. So you should loop through all the challenge objects in the array, and if they’re type: challenge, and provider: vanilla, and the url is empty, create that forum

Will there be only one object with type='challenge' and provider='vanilla'?

5. Are other types out of scope in this task?

[atelomycterus]

@jmgasper Don't have permissions to patch a challenge. Could you please ask Topcoder team to grant permissions?

[jmgasper]

1. Challenge without 'discussions'. 'discussions' is not required in the challenge schema.
   Should we create a group only in Vanilla?
   ![image](https://user-images.githubusercontent.com/10196632/94921513-20c65480-04c1-11eb-944c-c5b89beaaec5.png)

If we don't get a discussions object, we won't do anything. We don't need to create a group in that case.

2. Now the main category of challenge's discussions is named as a challenge's name.
   Should we use 'name' from a discussion object?
   {
   "provider": "vanilla",
   "name": "Justin Forum Test 11 Discussion",
   "type": "challenge"
   }

Yes, I think that makes sense

And you should assume that there can be multiple vanilla forums for whatever reason. So you should loop through all the challenge objects in the array, and if they’re type: challenge, and provider: vanilla, and the url is empty, create that forum

Will there be only one object with type='challenge' and provider='vanilla'?

I've asked Topcoder

1. Are other types out of scope in this task?

Yep, but if you run into any issues, let me know and I'll log things so we track them, thanks!

[atelomycterus]

@jmgasper Please apply PRs: topcoder-platform/challenge-forum-processor#31

Assumption

Implemented for 'one object with type='challenge' and provider='vanilla'
We will wait for a response from Topcoder team if we need support for multiple objects.
if yes then new questions arise. Should we create one group for all discussion objects or not?

Testing

I pushed the code. Couldn't test updating a challenge due to Forbidden Error.

[jmgasper]

@atelomycterus - The scopes have been updated to:

"scope": "all:challenges read:challenges write:challenges write:groups read:groups read:project read:user_profiles read:project-user read:resources read:terms read:project-members read:group",

[atelomycterus]

@jmgasper It isn't working. Got 'Not allowed to perfom this action!' for Put/Patch methods.

Vanilla config:

I tested with a generated token

Postman results with this token:
Get : https://api.topcoder-dev.com/v5/challenges/9d20c28a-2c06-432d-9b0c-ebdee23a9ded - OK

Put the same object - Not allowed

Patch with updated description - Not allowed