-
Notifications
You must be signed in to change notification settings - Fork 47
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
Create basic administrator dashboard with dummy overview statistics #392
Comments
See Starter UI Files for MVP for additional context about starter files. |
Grabbing this! |
@daaimah123 I'm making good progress on this ticket but I'm honestly not sure I'll be able to complete the whole thing by the end of sprint. Currently here's what I have:
|
@themagicianking great considerations and wow fantastic work so far!
|
@daaimah123 Thank you! I just linked the issue; I have sent @stmcpeters a message to see what thoughts they have on this since they are working on a similar issue and mentioned using frameworks as well. I will do some research and add another comment when that is done :) |
Potential CSS frameworks: Pros:
Cons:
Pros:
Cons:
I looked at quite a few other options but honestly I don't think any of them fit as well as these two, though I'm open to suggestions! |
@daaimah123 I have a question about the "applications by status" stats. I am assuming that, for each variable, you want to display applications that have ONLY gotten up to that point and NOT applications that have gotten to that point and beyond. That is, if 50% of applications have been submitted and of those 50%, 10% have also done the application workshop, "submitted" should display 40%, not 50%. Is that correct? Edit: upon closer inspection of the example graphics I think I may have gotten this backwards and am slightly more confused. I'm not sure if these are meant to be a guideline or not for the actual data aggregation. |
@themagicianking these are 100% guidelines and can be adjusted; this ticket is 100% about frontend for now and does not need to take into account the db work that will come later; leave a comment to where you land with that presentation as related to your dummy data and this issue Let's do custom styling for now without frameworks (leverage the existing css rules where able), let me know if you'd like to cut a sub-issue for that. |
Gotcha! That makes things a lot easier; I will focus on the CSS!
…On Thu, Jan 30, 2025 at 2:35 PM Daaimah Tibrey ***@***.***> wrote:
@themagicianking <https://github.com/themagicianking> these are 100%
guidelines and can be adjusted; this ticket is 100% about frontend for now
and does not need to take into account the db work that will come later;
leave a comment to where you land with that presentation as related to your
dummy data and this issue
Let's do custom styling for now without frameworks (leverage the existing
css rules where able), let me know if you'd like to cut a sub-issue for
that.
—
Reply to this email directly, view it on GitHub
<#392 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AVWG2DXN2E7WHFLPCXETDFD2NJ5JPAVCNFSM6AAAAABTQ4G7MKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMRVGQYDINRUGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
--
"There are still stars which move in ordered and beautiful rhythm. There
are still people in this world who keep promises."
*A Wind in the Door,* Madeleine L'Engle
|
@daaimah123 Just wanted to let you know that me and @kaylahrose have resolved the merge conflicts that were on this issue! I did have to create a new branch and cherry pick my old commits but it does seem to have worked correctly. I'm still unsure what exactly caused this as my develop branch is and was up to date at the time of rebasing, but it's working now. @ChasVanDav was/is having similar issues--I'm curious if anyone has any ideas on what the root issue of this could've been. |
@daaimah123 I wanted to give you an update on progress here--I'm really, really close to being done, but I'm honestly not sure I can finish this and my knowledge check today (I was not anticipating the rebase issues). Things I still have left to do: create the application review section & dynamically populate data instead of using hardcoded data. As of right now all aspects of the page are responsive. |
(Summarizing our conversation, @themagicianking ) Intended purpose of this ticket: track application status in one place, with visuals for data, and editing/updating Sub-Issues
Clarification / Discussion
|
@themagicianking can you give a status update on this related work? I see draft PR: Admin dashboard frontend and have attached it to this issue |
Problem:
Administrators need a centralized view of key platform statistics.
Implementation Details:
- Eligibility score
- Details (visual green checkmark for met eligibility category criteria, red "x" for ineligible, yellow warning icon for area that requires manual review, should list the provided data from applicant)
- WPM Score (eligibility is 50+)
- WPM Accuracy (eligibility is 80%)
- Two References (eligibility is both references details are all submitted)
- FreeCodeCamp Complete (warning icon, needs to be manually changed by admin user, eligibility is the complete single screenshot of all 115 problems, manual confirmation needs to be timestamped and added to admin only applicant notes with admin user or program staff role details)
- Gender (eligibility is not cis-male)
- Computer literate (eligibility is yes)
- Recent bootcamp (eligibility is no)
- Stable housing (eligibility is yes)
- Action
- leave admin-only note (under applicant's row)
- edit button
- message applicant (to be connected to applicant dashboard later)
- overlay popup form with option to edit populated fields
- timestamp into admin only notes detailing which application fields were changed
- overlay popup text edit form with option to write in a message
- timestamp into admin only notes under applicant's row
- this should be a dropdown field that span horizontally underneath the other applicant fields
- overlay popup text edit form with option to write in a message
- timestamp into admin only notes under applicant's row, detailing the message to applicant
Sample Dummy Data
Technical Concepts:
Acceptance Criteria:
mvp
branch, notdevelop
Example UI
The text was updated successfully, but these errors were encountered: