Skip to content

Pr 52: Improvement + Reimplementation #25

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
Merged
merged 32 commits into from
Mar 22, 2025
Merged

Pr 52: Improvement + Reimplementation #25

merged 32 commits into from
Mar 22, 2025

Conversation

@Jing27540
Copy link
Owner

@Jing27540 Jing27540 commented Mar 20, 2025

This PR covers the improvement/implementation of PR expertiza#52, focusing on enhancements related to the StudentTask view.

  1. Use Common Table component.
  2. Add Info icon and make it functionally.
  3. Remove uncessary comps, e.g. global searching bar.
  4. Format columns' data by creating global helper function, e.g. formatDate, caplizeFirstWord (make them reusable to all files)

prathyu99 and others added 30 commits April 16, 2024 20:55
@prathyu99
created a new page StudentTasks under student role and added a basic …
5a07b9f 
…table displaying the projects.
@prathyu99
Added link to navigate to new Student Task Detail page
e01eee6
@prathyu99
added checkbox for publishing rights
111ba6a
@prathyu99
added css for links
2ad4e8d
@soubhi
Adding Student Tasks Box
15c58c6
@soubhi
Merge pull request #1 from prathyu99/apiAndLayout
5c3de2c 
Adding Student Tasks Box
@soubhi
Modifying code to fetch data from dummy data
544b071
@soubhi
Merge pull request #2 from prathyu99/apiAndLayout
8a628a4 
Modifying code to fetch data from dummy data
@prathyu99
added icon i which displays text on hover
80b463c
@prathyu99
added css to adjust the layout
ab936ac
@soubhi
Modifying Revisions component and adding remaining days functionality
5e372f0
@soubhi
Merge branch 'main' into apiAndLayout
5c40ef2
@soubhi
Merge pull request #3 from prathyu99/apiAndLayout
ca8ac62 
Modifying Revisions component and adding remaining days functionality
@prathyu99
added comments to the code and resolved merge conflicts
1257334
@prathyu99
added comments to the code studenttasks
25f8ad2
@soubhi
Merge branch 'main' into main
19f3409
@prathyu99
added space in student task box, changed the hover text for publishin…
7c1a434 
…g rights, removed the badges column if it has no data.
@prathyu99
Merge remote-tracking branch 'origin/main'
a779945
@prathyu99
added space in student task box, changed the hover text for publishin…
e6cccb7 
…g rights, removed the badges column if it has no data.
@prathyu99
removed an unnecessary change
8bf0c02
@prathyu99
Merge branch 'expertiza:main' into main
66d950b
Merge branch 'main' into pr-52
906dcb1
Merge remote-tracking branch 'origin' into pr-52
af7256f
Commented out the overwritten table style within StudentTasks.module.css
53383c4
@cnayak99
Change the Student Task table using the Table component
d14f465
@cnayak99
Change the Student Task table using the Table component
4f6c0df
Start feature: Add info icon with pop-out comment placeholder
b0505b8
Update pop-out comments in Assignment table
2871438
@Jing27540
Merge pull request #24 from Jing27540/feature/info-icon-popout
f9327b7 
Great job!!
Finished the reimplementation of Course Assignment View.
39e1593
@Jing27540 Jing27540 requested review from DrYenLo and cnayak99 March 20, 2025 19:07
@Jing27540 Jing27540 changed the title Pr 52 Pr 52: Improvement + Reimplementation Mar 20, 2025
@Jing27540 Jing27540 merged commit 74e6f13 into main Mar 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cnayak99 cnayak99 Awaiting requested review from cnayak99

@DrYenLo DrYenLo Awaiting requested review from DrYenLo

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Jing27540 @prathyu99 @soubhi @cnayak99