Skip to content

Removed text from rules banner.#211

Open
stephenbit wants to merge 4 commits into
masterfrom
Remove-rules-from-banner
Open

Removed text from rules banner.#211
stephenbit wants to merge 4 commits into
masterfrom
Remove-rules-from-banner

Conversation

@stephenbit

Copy link
Copy Markdown
Collaborator

No description provided.

@stephenbit stephenbit left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Removed the css file as there was only 2 style selectors left there.

return $(".navbar").$(".heading");
}
get headlineBanner() {
return $(".headline-banner");

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

There is now no selector .headline-banner

import Row from "react-bootstrap/Row";
import Col from "react-bootstrap/Col";

const RouteMapRules = () => {

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Updated the name because it no longer contains any rules.

@stephenbit stephenbit changed the title Removed text from rules banner. See notes. Removed text from rules banner. Sep 5, 2021
@sonarqubecloud

sonarqubecloud Bot commented Sep 6, 2021

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

import Row from "react-bootstrap/Row";
import Col from "react-bootstrap/Col";

const HeadlineBanner = () => {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The only thing I'd suggest is that if a component doesn't do anything, it would be better to inline the static formatting and bin the component.

@donal-s

donal-s commented Sep 7, 2021

Copy link
Copy Markdown
Collaborator

I got a failure on npm start here:
`
STACovidDashboard/reactDashboard/node_modules/watchpack/lib/chokidar.js:17
throw new Error(
^

Error: No version of chokidar is available. Tried chokidar@2 and chokidar@3.
You could try to manually install any chokidar version.
chokidar@3: Error: Cannot find module 'chokidar'
`

The build failure happens on master too, so likely the issue is me. If npm start works for you, then don't worry about it :)

@donal-s donal-s left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Minor comment

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.

2 participants