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

Issue #152: Better error messages in query view #153

Merged
merged 1 commit into from
Jan 10, 2024

Conversation

reckart
Copy link
Member

@reckart reckart commented Jan 10, 2024

What's in the PR

  • Check for proper data location and type system location and generate error otherwise

How to test manually

  • Try entering an invalid location into the data location field
  • Try running a query without specifying a type system

Automatic testing

  • PR adds/updates unit tests

Documentation

  • PR adds/updates documentation

Organizational

  • PR includes new dependencies.
    Only dependencies under approved licenses are allowed. LICENSE and NOTICE files in the respective modules where dependencies have been added as well as in the project root have been updated.

@reckart reckart added the ⭐️ Enhancement New feature or request label Jan 10, 2024
@reckart reckart added this to the 3.4.0 milestone Jan 10, 2024
@reckart reckart self-assigned this Jan 10, 2024
- Check for proper data location and type system location and generate error otherwise
@reckart reckart force-pushed the feature/152-Better-error-messages-in-query-view branch from b2cff37 to ae7e806 Compare January 10, 2024 11:56
@reckart reckart changed the title #152 - Better error messages in query view Issue #152: Better error messages in query view Jan 10, 2024
@reckart reckart merged commit 3d2d94d into main Jan 10, 2024
1 check passed
@reckart reckart deleted the feature/152-Better-error-messages-in-query-view branch January 10, 2024 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⭐️ Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant