Skip to content

refactor: log error details when loading ERD data - #5

Open
jemartinezrdz wants to merge 1 commit into
devIAfrom
codex/refactorizar-bloque-catch-en-loaderddata
Open

jemartinezrdz wants to merge 1 commit into
devIAfrom
codex/refactorizar-bloque-catch-en-loaderddata

Conversation

@jemartinezrdz

Copy link
Copy Markdown
Owner

Summary

  • log ERD data load failures in ERDViewer

Testing

  • npm test (fails: Missing script: "test")
  • npm run lint
  • npm run type-check

https://chatgpt.com/codex/tasks/task_e_68916d65d3388324a3ad65edff016563

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR improves error handling in the ERDViewer component by adding detailed error logging when ERD data loading fails. The change replaces an unused error parameter with proper error logging to aid in debugging.

  • Enhanced error handling by logging error details to the console
  • Replaced unused error parameter with actual error logging implementation
Comments suppressed due to low confidence (1)

autodocops-ui/src/components/ERDViewer.tsx:97

  • [nitpick] The variable name 'error' is generic. Consider using a more descriptive name like 'loadError' or 'erdLoadError' to better indicate the context of this error.
    } catch (error) {

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants