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

Add an Info Alert when there's nothing to review #2492

Conversation

flacial
Copy link
Member

@flacial flacial commented Nov 5, 2022

Closes #2462

Description

Add an info to inform the user that there's nothing to review at the moment.

Image image

Testing

  1. Go to /review/js1
  2. Make sure it displays All the submissions were reviewed. Come and check later!
  3. Go to /review/js0
  4. Make sure it shows the submissions

@vercel
Copy link

vercel bot commented Nov 5, 2022

@flacial is attempting to deploy a commit to the c0d3-prod Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Nov 5, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
c0d3-app ✅ Ready (Inspect) Visit Preview Nov 5, 2022 at 0:08AM (UTC)

@codecov
Copy link

codecov bot commented Nov 5, 2022

Codecov Report

Merging #2492 (8f4a757) into master (0f9e90f) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #2492   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          186       186           
  Lines         3322      3324    +2     
  Branches       881       883    +2     
=========================================
+ Hits          3322      3324    +2     
Impacted Files Coverage Δ
pages/review/[lesson].tsx 100.00% <100.00%> (ø)

@flacial flacial merged commit ab65256 into garageScript:master Nov 5, 2022
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.

Lack of information when there's nothing to review
2 participants