Skip to content
This repository was archived by the owner on Jul 6, 2020. It is now read-only.

Add a soft check when editing phase details #265

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

nikochiko
Copy link

@nikochiko nikochiko commented Jan 7, 2020

As part of a Google Code-In task.

Changes proposed in this pull request:

  • Include a soft-check to ensure that the host does not enter:
    max submissions/day > max number of submissions/month > Max total submissions

As part of a Google Code-In task.

#### Changes proposed:
* Include a soft-check so that:
max submissions/day > max number of submissions/month > Max total submissions
@codecov-io
Copy link

codecov-io commented Jan 7, 2020

Codecov Report

Merging #265 into master will decrease coverage by 0.1%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #265      +/-   ##
==========================================
- Coverage   50.62%   50.51%   -0.11%     
==========================================
  Files          66       66              
  Lines        3761     3771      +10     
  Branches      442      444       +2     
==========================================
+ Hits         1904     1905       +1     
- Misses       1762     1771       +9     
  Partials       95       95
Impacted Files Coverage Δ
...ephases/editphasemodal/editphasemodal.component.ts 33.76% <0%> (-5.04%) ⬇️
...bliclists/challengelist/challengelist.component.ts 47.61% <0%> (+0.95%) ⬆️
Impacted Files Coverage Δ
...ephases/editphasemodal/editphasemodal.component.ts 33.76% <0%> (-5.04%) ⬇️
...bliclists/challengelist/challengelist.component.ts 47.61% <0%> (+0.95%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e81fb10...2a90aa2. Read the comment docs.

Copy link
Member

@Ram81 Ram81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nikochiko changes look good to me, can you share screenshot of the working?

@nikochiko
Copy link
Author

nikochiko commented Jan 8, 2020

@Ram81 I am facing this issue for the dev build 😕 https://github.com/Cloud-CV/EvalAI-ngx/issues/249. Can you please share if there is any other way to test the working?

(Not linking issue directly as it could create reference to the unrelated issue)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants