daily-code
daily-code copied to clipboard
code100x
feature: List all DSA Problem Statements in the admin dashboard
Note: Please only raise the PR for this once https://github.com/code100x/daily-code/pull/203 is reviewed and merged, since it is dependent on that.
- Add one more Tab for viewing the existing PS, integrate this with the current admin dashboard flow.
- Add a way for admins to edit those PS.
I think listing and editing of the problems are already there.
This is right?
Or you mean for the dsa thing?
Will go throught schema again and will get back to you.
Working on this. Will update within tonight. Have some refactoring for the imports of other data in the other admin tabs as well.
https://github.com/code100x/daily-code/assets/108384868/da1a2656-6fce-4d8f-a4a9-bf526d2c12ac
Working on this. Suggessions?
The test cases are based on the argument names provided, are you checking that.
I think you can just use the same form I've created to edit the PS.
The test cases are based on the argument names provided, are you checking that.
I think you can just use the same form I've created to edit the PS.
This right?
The test cases are based on the argument names provided, are you checking that.
I think you can just use the same form I've created to edit the PS.
Why can't we take the thing you done to the admin page. So everything will be in a single place?
Yeah you can. I moved that to a separate page because it was a big form. Would have looked too cluttered on this page, specially when you add a large number of test cases.
In future once more languages are added, and if more things are needed it would be decluttered that way.
A good thing I think of is making this whole problem component a fullscreen dialog. So we can use this for editing and adding things. Without making things move crazyly in the page. Also can implement the JSON feature in the dialog itself.
can we also take parameter type in the input because for other languages we would need those
can we also take parameter type in the input because for other languages we would need those
Need a bit more arch changes. Like we should have some more thing to the db then. Will figure this next