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

Datasource errors blocking full creation steps #12127

Merged
merged 3 commits into from
Oct 20, 2023

Conversation

mike12345567
Copy link
Collaborator

@mike12345567 mike12345567 commented Oct 20, 2023

Description

Fixing datasource error states causing a datasource to fail to be built fully - this was simply a frontend issue of the error not being handled on creation.

Addresses: #12121 and https://linear.app/budibase/issue/BUDI-7650/if-an-external-data-source-is-added-were-any-of-the-tables-within-dont

Feature branch env

Feature Branch Link

…lt fully - this was simply a frontend issue of the error not being handled on creation.
@mike12345567 mike12345567 self-assigned this Oct 20, 2023
@mike12345567 mike12345567 added the firestorm Data/Infra/Revenue Team label Oct 20, 2023
Copy link
Collaborator

@adrinr adrinr left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov-commenter
Copy link

Codecov Report

Merging #12127 (dbf5a9c) into master (90cdda7) will not change coverage.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@           Coverage Diff           @@
##           master   #12127   +/-   ##
=======================================
  Coverage   57.36%   57.36%           
=======================================
  Files         259      259           
  Lines        7953     7953           
  Branches     1483     1483           
=======================================
  Hits         4562     4562           
  Misses       3107     3107           
  Partials      284      284           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@mike12345567 mike12345567 merged commit 89d6486 into master Oct 20, 2023
10 checks passed
@mike12345567 mike12345567 deleted the fix/12121-datasource-error-stopping-creation branch October 20, 2023 12:14
@github-actions github-actions bot locked and limited conversation to collaborators Oct 20, 2023
@samwho
Copy link
Collaborator

samwho commented Oct 21, 2023

Thank you for sorting this 🙏

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
firestorm Data/Infra/Revenue Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants