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(notebook): More specific error in query inspector #72

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

Catalin-web
Copy link
Contributor

@Catalin-web Catalin-web commented Oct 1, 2024

Pull Request

🀨 Rationale

Bug [NotebookPlugin] A generic error is exposed in UI

πŸ‘©β€πŸ’» Implementation

Added exception, errorCode on the execution

πŸ§ͺ Testing

image

βœ… Checklist

src/datasources/notebook/datasource.ts Outdated Show resolved Hide resolved
@Catalin-web Catalin-web merged commit 90bb8e9 into main Oct 2, 2024
1 check passed
mure pushed a commit that referenced this pull request Oct 2, 2024
## [3.14.1](v3.14.0...v3.14.1) (2024-10-02)

### Bug Fixes

* **notebook:** More specific error in query inspector ([#72](#72)) ([90bb8e9](90bb8e9))
@mure
Copy link
Collaborator

mure commented Oct 2, 2024

πŸŽ‰ This PR is included in version 3.14.1 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

@mure mure added the released label Oct 2, 2024
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.

5 participants