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

Remove React Warnings found in tests #72

Open
AutomatedTester opened this issue Jun 24, 2024 · 4 comments
Open

Remove React Warnings found in tests #72

AutomatedTester opened this issue Jun 24, 2024 · 4 comments
Labels
good first issue Good for newcomers

Comments

@AutomatedTester
Copy link
Member

There are a number of warnings from React components e.g.

[browser] Warning: React does not recognize the `%s` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `%s` instead. If you accidentally passed it from a parent component, remove it from the DOM element.%s lineHight linehight 
    at p
    at O2 (http://localhost:39113/node_modules/.vite/deps/styled-components.js?v=b6eaafa4:1249:6)
    at div
    at O2 (http://localhost:39113/node_modules/.vite/deps/styled-components.js?v=b6eaafa4:1249:6)
    at footer
    at O2 (http://localhost:39113/node_modules/.vite/deps/styled-components.js?v=b6eaafa4:1249:6)
    at Footer
    at FooterComponent
    ```
    
 Issues can be found by doing `npm run test` once the repo has been checked out
@AutomatedTester AutomatedTester added the good first issue Good for newcomers label Jun 24, 2024
@alwaysahustler
Copy link

Working on it

@blueewhitee
Copy link

blueewhitee commented Jul 27, 2024

removed warnings that were coming from react components

Screencast.from.28-07-24.12.38.58.AM.IST.webm

prop was being passed down to DOM elements. So used prefix"$" to indicate that they are for styling purposes only.

Used npm run test command as recommended.

@hustlerZzZ
Copy link

Hey, Can i fix this ?

@VanshikaSabharwal
Copy link

@AutomatedTester i can fix this warning issue. Please assign it to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants