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.

Sign up for GitHub

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

Jump to bottom

Tech Debt: Remove console.log Statement in BaseGraph.ts #217

Closed
jenny-s51 opened this issue Jun 7, 2024 · 0 comments · Fixed by #226
Closed

Tech Debt: Remove console.log Statement in BaseGraph.ts #217

jenny-s51 opened this issue Jun 7, 2024 · 0 comments · Fixed by #226
Assignees
@jenny-s51

Comments

@jenny-s51
Copy link
Collaborator

jenny-s51 commented Jun 7, 2024 •
edited
Loading

Describe the problem
This line in BaseGraph.ts is causing unexpected print statements in the RHOAI dev console.

How do you reproduce the problem?
See opendatahub-io/odh-dashboard#2886 (review)
@jenny-s51 jenny-s51 self-assigned this Jun 7, 2024
@github-project-automation github-project-automation bot moved this to Needs triage in PatternFly Issues Jun 7, 2024
@jenny-s51 jenny-s51 mentioned this issue Jun 7, 2024
Merged
7 tasks
@jenny-s51 jenny-s51 changed the title Tech Debt: Remove Stray console.log Statement in BaseGraph.ts Tech Debt: Remove console.log Statement in BaseGraph.ts Jun 10, 2024
@nicolethoen nicolethoen moved this from Needs triage to In Progress in PatternFly Issues Jun 28, 2024
lordrip added a commit to lordrip/react-topology that referenced this issue Jul 18, 2024
@lordrip
fix(graph): Remove console.log() statement
7477c02 
fix: patternfly#217
@lordrip lordrip mentioned this issue Jul 18, 2024
Merged
1 task
@github-project-automation github-project-automation bot moved this from In Progress to Done in PatternFly Issues Jul 23, 2024
jeff-phillips-18 pushed a commit to jeff-phillips-18/react-topology that referenced this issue Oct 25, 2024
@lordrip @jeff-phillips-18
fix(graph): Remove console.log() statement (patternfly#226)
970574f 
fix: patternfly#217
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jenny-s51 jenny-s51

Labels
None yet
Projects
PatternFly Issues
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(graph): Remove console.log() statement lordrip/react-topology
1 participant
@jenny-s51