Conversation
Fixed CSS style issue
PS-423 - fix retUrl in gigs modal
PS-412 - replace taas email
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| } = props; | ||
| let retUrl; | ||
| // eslint-disable-next-line no-restricted-globals | ||
| const retUrl = isomorphy.isClientSide() ? location.href : null; |
There was a problem hiding this comment.
[❗❗ security]
Using location.href directly can lead to security issues, such as open redirects, if not handled carefully. Consider using a safer method to construct URLs or validate the URL before use.
| color: $tc-orange; | ||
| } | ||
|
|
||
| .data-science-accent-color { |
There was a problem hiding this comment.
[maintainability]
The addition of .data-science-accent-color seems redundant since it duplicates the existing .datasci-accent-color with the same style. Consider removing one to avoid unnecessary duplication, which can lead to maintenance challenges.
| color: $tc-orange; | ||
| } | ||
|
|
||
| .data-science-accent-color { |
There was a problem hiding this comment.
[maintainability]
The addition of .data-science-accent-color seems redundant as it duplicates the existing .datasci-accent-color class. Consider removing one to avoid confusion and maintain consistency.
3 participants
https://topcoder.atlassian.net/browse/PS-412
https://topcoder.atlassian.net/browse/PS-423
https://topcoder.atlassian.net/browse/PS-411