Skip to content
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.

Sign up for GitHub

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

Jump to bottom

FEATURE: Added Admin panel for Coding Problems #295

Merged
merged 23 commits into from
May 12, 2024
Merged

FEATURE: Added Admin panel for Coding Problems #295

merged 23 commits into from
May 12, 2024

Conversation

SujithThirumalaisamy
Copy link
Contributor

@SujithThirumalaisamy SujithThirumalaisamy commented Apr 16, 2024 •
edited
Loading

PR Fixes:

Done:

  • Listing the ProblemStatements
  • Adding the form for editing ProblemStatement
  • Add query for the testcases
  • Button to create new Problem Statement
  • Add Types
  • Add the Json import feature

Pending:

  • UI Improvement ( Someone feel free to add this )
2024-04-18.10-15-03.mp4
2024-04-18.11-23-31.mp4

@SujithThirumalaisamy SujithThirumalaisamy marked this pull request as ready for review April 18, 2024 05:55
@SujithThirumalaisamy SujithThirumalaisamy changed the title Added the form for problem-satements FEATURE: Added Admin panel for Coding Problems Apr 21, 2024
@SujithThirumalaisamy
Copy link
Contributor Author

SujithThirumalaisamy commented Apr 26, 2024 •
edited
Loading

TODO:

  • Migrate the create problem with the type of Code to the problem statement's tab
  • Also add feature to import a CSV

Note: Will add this in the next PR most probably

@SujithThirumalaisamy
Copy link
Contributor Author

2024-04-29.01-22-35.mp4

CSV import Feature

@SujithThirumalaisamy
Copy link
Contributor Author

All Done And Ready For Review. @hkirat

@SujithThirumalaisamy SujithThirumalaisamy deleted the admin-problem-statement branch May 4, 2024 20:38
@SujithThirumalaisamy SujithThirumalaisamy restored the admin-problem-statement branch May 4, 2024 20:39
@SujithThirumalaisamy SujithThirumalaisamy mentioned this pull request May 7, 2024
Merged
8 tasks
@SujithThirumalaisamy
Copy link
Contributor Author

@hkirat Resolved the conflicts. Can be reviewed

@SujithThirumalaisamy
Copy link
Contributor Author

Resolved Conflicts

hkirat
hkirat reviewed May 12, 2024
View reviewed changes
apps/web/components/utils.tsx
}

export async function updateProblemStatement(problemStatementId: string, data: any) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

anys are bad

hkirat
hkirat reviewed May 12, 2024
View reviewed changes
packages/store/src/atoms/index.ts
export * from "./profileSidebar";
export * from "./profileSidebar";
Copy link
Collaborator

Choose a reason for hiding this comment

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

why twice?

@SujithThirumalaisamy
Copy link
Contributor Author

Thats just a sample thing. Not really thinked about it. ( twosum return that)

@SujithThirumalaisamy
Copy link
Contributor Author

Test cases are migrated to github. In the upcoming execution engine. Will change that in another pr

@hkirat
Copy link
Collaborator

hkirat commented May 12, 2024

/bounty $69

@hkirat hkirat merged commit 6beb890 into code100x:main May 12, 2024
1 check failed
@SujithThirumalaisamy
Copy link
Contributor Author

Will fix the any in a pr. Have a lot of any in the utils

@SujithThirumalaisamy SujithThirumalaisamy deleted the admin-problem-statement branch May 12, 2024 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hkirat hkirat hkirat left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SujithThirumalaisamy @hkirat