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

Mobile responsive website #153

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

Conversation

premalrupnur
Copy link
Collaborator

@premalrupnur premalrupnur commented Dec 20, 2020


This change is Reviewable

@gitpod-io
Copy link

gitpod-io bot commented Dec 20, 2020

@ghost
Copy link

ghost commented Dec 20, 2020

DeepCode's analysis on #33d361 found:

  • ℹ️ 2 minor issues. 👇

Top issues

Description Example fixes
Mixed spaces and tabs. Occurrences: 🔧 Example fixes
React identifier typo: change class to className. Occurrences: 🔧 Example fixes

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@lgtm-com
Copy link

lgtm-com bot commented Dec 20, 2020

This pull request introduces 1 alert when merging a07b244 into b348219 - view on LGTM.com

new alerts:

  • 1 for Unused or undefined state property

@lgtm-com
Copy link

lgtm-com bot commented Dec 21, 2020

This pull request introduces 2 alerts when merging 2e373c1 into b348219 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class
  • 1 for Unused or undefined state property

@codecakes codecakes requested review from ksr89 and Tapudp December 21, 2020 16:53
@lgtm-com
Copy link

lgtm-com bot commented Dec 21, 2020

This pull request introduces 2 alerts when merging 2835cd6 into b348219 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class
  • 1 for Unused or undefined state property

@lgtm-com
Copy link

lgtm-com bot commented Dec 22, 2020

This pull request introduces 2 alerts when merging 4cb676d into b348219 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class
  • 1 for Unused or undefined state property

@lgtm-com
Copy link

lgtm-com bot commented Dec 22, 2020

This pull request introduces 2 alerts when merging 83814e9 into b348219 - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class
  • 1 for Unused or undefined state property

@lgtm-com
Copy link

lgtm-com bot commented Jan 4, 2021

This pull request introduces 2 alerts when merging 6454ddc into c1c868b - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class
  • 1 for Unused or undefined state property

@lgtm-com
Copy link

lgtm-com bot commented Jan 13, 2021

This pull request introduces 2 alerts when merging e5e7121 into c1c868b - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class
  • 1 for Unused or undefined state property

@premalrupnur
Copy link
Collaborator Author

@codecakes its almost done need the reviews so that I can work on those minor changes

@lgtm-com
Copy link

lgtm-com bot commented Jan 13, 2021

This pull request introduces 2 alerts when merging 33d361b into c1c868b - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class
  • 1 for Unused or undefined state property

@codecakes
Copy link
Contributor

@PunitSoniME can u pick this up against develop branch?

@PunitSoniME
Copy link

@PunitSoniME can u pick this up against develop branch?

Sure

@codecakes
Copy link
Contributor

@PunitSoniME can u pick this up against develop branch?

Sure

did you get a chance to look at this?

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.

3 participants