Skip to content
This repository was archived by the owner on Jan 23, 2025. It is now read-only.

fix: groups, terms association lost #78

Conversation

rakibansary
Copy link
Contributor

  • if v4 challenge api returns 404 immediately after creation of the challenge when replaying the event
  • groups and other data are lost; resolve this problem by adding those data without attempting to get
  • challenge details from v4 as required info to make those associations is already present

* if v4 challenge api returns 404 immediately after creation of the challenge when replaying the event
* groups and other data are lost; resolve this problem by adding those data without attempting to get
* challenge details from v4 as required info to make those associations is already present
@LieutenantRoger LieutenantRoger changed the base branch from develop to fix/association-loss April 29, 2022 14:10
@LieutenantRoger LieutenantRoger merged commit e9b9863 into topcoder-archive:fix/association-loss Apr 29, 2022
@@ -224,10 +225,10 @@ async function associateChallengeTerms (v5Terms, legacyChallengeId, createdBy, u
const standardTerms = _.find(v5Terms, e => e.id === config.V5_TERMS_STANDARD_ID)
const legacyStandardTerms = _.find(legacyTermsArray, e => _.toNumber(e.id) === _.toNumber(config.LEGACY_TERMS_STANDARD_ID))

logger.debug(`NDA: ${config.V5_TERMS_NDA_ID} - ${JSON.stringify(nda)}`)
Copy link
Contributor

Choose a reason for hiding this comment

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

May I know why the log is commented out ? @rakibansary

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

Successfully merging this pull request may close these issues.

2 participants