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 frontend console errors #6421

Merged
merged 2 commits into from
Jan 22, 2024
Merged

Conversation

Richard-Pentecost
Copy link
Contributor

Description of change

Fix console warnings

Test instructions

What should I see?

Screenshots

Before

Add a screenshot

After

Add a screenshot

Checklist

  • Has the branch been rebased to main?
  • Automated tests (Any of the following when applicable: Unit, Functional or End-to-End)
  • Manual compatibility testing (Browsers: Chrome, Firefox, Edge, Safari)

@Richard-Pentecost Richard-Pentecost requested a review from a team as a code owner January 10, 2024 11:57
Copy link

cypress bot commented Jan 10, 2024

Passing run #50280 ↗︎

0 24 0 0 Flakiness 0

Details:

add taskProps to ProjectInteraction.jsx to prevent errors
Project: data-hub-frontend Commit: 15f7caf0d0
Status: Passed Duration: 02:00 💡
Started: Jan 18, 2024 2:01 PM Ended: Jan 18, 2024 2:03 PM

Review all test suite changes for PR #6421 ↗︎

@Richard-Pentecost Richard-Pentecost force-pushed the firebreak/console-warnings branch from 652d31b to 138e3a7 Compare January 18, 2024 12:12
@Richard-Pentecost Richard-Pentecost force-pushed the firebreak/console-warnings branch from 138e3a7 to d758a18 Compare January 18, 2024 12:16
@Richard-Pentecost Richard-Pentecost force-pushed the firebreak/console-warnings branch from 252a727 to 15f7caf Compare January 18, 2024 13:57
@Richard-Pentecost Richard-Pentecost merged commit e0aaf4b into main Jan 22, 2024
16 checks passed
@Richard-Pentecost Richard-Pentecost deleted the firebreak/console-warnings branch January 22, 2024 16:15
chopkinsmade pushed a commit that referenced this pull request Jan 22, 2024
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.

4 participants