-
Notifications
You must be signed in to change notification settings - Fork 21
[PROD] - Security, clean up & bug fixes #1368
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.
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
Merged
Changes from 33 commits
Commits
Show all changes
46 commits
Select commit
Hold shift + click to select a range
5ce15aa
Cleanup old code for the review scorecard ui
vas3a 3b1193a
fix: allow only admins and reviewer to respond to appeals
hentrymartin 0361d8c
fix: lint
hentrymartin 985e16a
Merge pull request #1361 from topcoder-platform/pm-3141
hentrymartin 1468bd4
fix: filter submissions to design challenges if its configured that way
hentrymartin 57b48d3
fix: can view submissions
hentrymartin 6528746
fix: lint
hentrymartin d37b511
fix: lint
hentrymartin 195226c
fix: lint
hentrymartin d088964
Merge pull request #1364 from topcoder-platform/pm-2662_1
hentrymartin 951de7d
Merge pull request #1359 from topcoder-platform/cleanup-old-review-sc…
kkartunov c3baec0
point tc deps to master & npm minor update
kkartunov 65eec2d
ci: deploy to dev env
kkartunov d6462df
back to auth-lib-v2
kkartunov da81018
fix: directly open scorecard in edit mode
hentrymartin 5a25ed2
fix: revalidate appeals and response
hentrymartin d216fde
fix: revalidate appeals and response
hentrymartin b111246
fix: lint
hentrymartin 548b0c4
fix: lint
hentrymartin 7fd1d25
fix: restrict reviews in review tab for design challenge with config …
hentrymartin b6ed3ed
fix: lint
hentrymartin 995d81e
fix: lint
hentrymartin 75664d7
update yarn.lock
kkartunov 2e1fc7d
Merge pull request #1367 from topcoder-platform/pm-2662_2
kkartunov 0563cf0
fix trivy alerts
kkartunov b8d39fa
Merge branch 'dev' into security
kkartunov 48816c5
ci: remove from dev
kkartunov 3db7d1a
Merge pull request #1365 from topcoder-platform/security
kkartunov 61ad75b
security fixes - high
kkartunov e564eed
Merge pull request #1369 from topcoder-platform/security
kkartunov 5b01f5f
drop ssl serts
kkartunov 72028ad
Merge pull request #1370 from topcoder-platform/security
kkartunov 1b0df87
Merge pull request #1366 from topcoder-platform/pm-3141_1
kkartunov 21226f2
fix: repond to appeals
hentrymartin 12c4c64
fix: repond to appeals
hentrymartin d0a5e6d
fix: respond to appeals
hentrymartin 86b03c8
fix: lint
hentrymartin 4eb2201
fix: show only my winning results
hentrymartin 8fac84b
fix: lint
hentrymartin 5163510
Merge pull request #1371 from topcoder-platform/pm-3141_3
hentrymartin 1ced8a1
Merge pull request #1372 from topcoder-platform/pm-2662_3
hentrymartin e9d4976
fix: show only submissions with passed screening score
hentrymartin 6ae3ec0
fix: lint
hentrymartin d1cf5c6
Tweak this path to address 401 errors in the service
jmgasper 23a62d1
Merge pull request #1373 from topcoder-platform/pm-2573
hentrymartin 9ed7544
Merge pull request #1375 from topcoder-platform/master
kkartunov File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[⚠️
maintainability]Using
@ts-expect-errorto suppress TypeScript errors should be avoided if possible, as it can hide potential issues. Investigate whyReactMarkdowncannot be used as a JSX component and address the root cause. This will improve maintainability and ensure type safety.