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.

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

Revamped SubmissionListNavigator #117

Merged
merged 4 commits into from
Jul 18, 2024
Merged

Conversation

qhanson55
Copy link
Collaborator

@qhanson55 qhanson55 commented Jul 16, 2024

Description

Revamped SubmissionListNavigator based on criteria in the comment (11:59am) in PADS-189.

Types of changes

Feature, added visibility and functionality to the tabes fields to greater help Navigators utilize the Submissions List.

Checklist

  • I have read the CONTRIBUTING doc
  • I have checked that unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

Copy link

Coverage Report (Application)

Totals Coverage
Statements: 45.45% ( 918 / 2020 )
Methods: 35.5% ( 131 / 369 )
Lines: 56.37% ( 597 / 1059 )
Branches: 32.09% ( 190 / 592 )

Copy link

github-actions bot commented Jul 16, 2024

Coverage Report (Frontend)

Totals Coverage
Statements: 32.49% ( 1421 / 4374 )
Methods: 27.87% ( 245 / 879 )
Lines: 37.17% ( 878 / 2362 )
Branches: 26.3% ( 298 / 1133 )

Copy link

codeclimate bot commented Jul 16, 2024

Code Climate has analyzed commit c94a3bf and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 0.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 43.1% (0.1% change).

View more on Code Climate.

@Subin1Doo
Copy link

Everything is looking good! Just one minor comment, First name heading is split into two rows, can we put it into one like the rest? Thanks!

@qhanson55 qhanson55 force-pushed the feature/revamp-submissions-view branch from 0431344 to 20a99d5 Compare July 16, 2024 15:55
@qhanson55
Copy link
Collaborator Author

Everything is looking good! Just one minor comment, First name heading is split into two rows, can we put it into one like the rest? Thanks!

Fixed! Nice catch

sanjaytkbabu
sanjaytkbabu previously approved these changes Jul 16, 2024
@qhanson55 qhanson55 force-pushed the feature/revamp-submissions-view branch from 20a99d5 to db52c3f Compare July 17, 2024 18:18
@qhanson55 qhanson55 requested a review from sanjaytkbabu July 17, 2024 19:30
@qhanson55
Copy link
Collaborator Author

I have combined PADS-207 & PADS-220 as the bug is related to this PR and is only replicated here.

@qhanson55 qhanson55 dismissed sanjaytkbabu’s stale review July 17, 2024 19:32

added another ticket/bigfix to PR

@qhanson55 qhanson55 force-pushed the feature/revamp-submissions-view branch from 2cb62cb to d487adb Compare July 17, 2024 20:12
@wilwong89 wilwong89 merged commit b0bd1fa into master Jul 18, 2024
18 of 19 checks passed
@kyle1morel kyle1morel deleted the feature/revamp-submissions-view branch August 14, 2024 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants