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

Fix styling error for response text #395

Merged
merged 2 commits into from
Apr 6, 2020

Conversation

HemanshuGandhi
Copy link

Part of Hemanshu's deliverables for #380

@HemanshuGandhi HemanshuGandhi requested a review from a team April 6, 2020 14:50
@HemanshuGandhi HemanshuGandhi self-assigned this Apr 6, 2020
@HemanshuGandhi HemanshuGandhi added this to the v1.4 milestone Apr 6, 2020
@netlify
Copy link

netlify bot commented Apr 6, 2020

Deploy preview for notably-app ready!

Built with commit d9fa7a5

https://deploy-preview-395--notably-app.netlify.com

@HemanshuGandhi HemanshuGandhi added the type.Enhancement An enhancement to an existing story label Apr 6, 2020
@codecov
Copy link

codecov bot commented Apr 6, 2020

Codecov Report

Merging #395 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #395   +/-   ##
=========================================
  Coverage     66.76%   66.76%           
  Complexity      509      509           
=========================================
  Files            97       97           
  Lines          1980     1980           
  Branches        195      195           
=========================================
  Hits           1322     1322           
  Misses          579      579           
  Partials         79       79           

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 017dbfa...d9fa7a5. Read the comment docs.

Copy link

@kevinputera kevinputera left a comment

Choose a reason for hiding this comment

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

LGTM

@HemanshuGandhi HemanshuGandhi merged commit ea6b338 into AY1920S2-CS2103T-W17-2:master Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.High Must do type.Enhancement An enhancement to an existing story
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants