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

feat: query result sharing #904

Closed
wants to merge 4 commits into from
Closed

Conversation

AdiGajbhiye
Copy link
Collaborator

@AdiGajbhiye AdiGajbhiye commented Feb 14, 2024

Overview

  • Added new functionality to enable query sharing, allowing users to upload query preview results to the Altimate frontend.
  • Added a new boolean property dbt.enableQuerySharing to the package.json file with a default value of false.
Screenshot 2024-02-21 at 1 07 07 PM

Checklist

  • I have run this code and it appears to resolve the stated issue
  • README.md updated and added information about my change

@AdiGajbhiye AdiGajbhiye requested review from suryaiyer95, mdesmet and saravmajestic and removed request for suryaiyer95 February 14, 2024 20:21
Copy link
Collaborator

@saravmajestic saravmajestic left a comment

Choose a reason for hiding this comment

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

Just minor text change. codewise it looks fine. I dont have any idea about the functionality.

.get<string>("altimateInstanceName");
if (!company) {
window.showErrorMessage(
"Please instance name and API key for inviting user.",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"Please instance name and API key for inviting user.",
"Please add instance name and API key for inviting user.",

Copy link
Collaborator

Choose a reason for hiding this comment

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

why dont you use handlePreviewFeatures check instead of checking just instance name?

@AdiGajbhiye AdiGajbhiye force-pushed the feat/query-result-sharing branch 3 times, most recently from f8bba39 to 53c1845 Compare February 20, 2024 18:23
feat: change endpoint

feat: added button

WIP

WIP

WIP

WIP

feat: added checks

WIP

fix: styling

WIP

feat: added invite user

feat: added feature flag in settings

WIP

feat: added progress bars

fix: styling

WIP

WIP

fix: added greyed out share button

feat: add tooltip

WIP

WIP

WIP

WIP

WIP

fix: text

fix: error message

fix: model path

fix: disable admin role

fix: as per comments

fix: text

fix: make url a linik

feat: added project name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants