Skip to content

Conversation

@edugfilho
Copy link
Contributor

@edugfilho edugfilho commented Sep 19, 2025

Link also opens in a new tab now.

Pull Request checklist

  • The pull request has a descriptive title (and a reference to an issue it
    fixes, if applicable)
  • All tests and linter checks are passing
  • The pull request is free of merge conflicts

import tippy from "./tippy";
export let href;
export let target = "";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this makes the component open the link in a new tab

sqlContent={getGleanPingQuery(selectedAppVariant.table)}
/>
</td>
<tr>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

added a here to fix a broken layout

@edugfilho edugfilho requested a review from badboy September 19, 2025 17:33
@edugfilho edugfilho removed the request for review from badboy September 19, 2025 19:19
Copy link
Member

@badboy badboy left a comment

Choose a reason for hiding this comment

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

At first I was confused as the screenshot didn't show that any of the "show sql" was left, but a latter commit added that back, so I guess the screenshot is outdated.

r+! Though CI is now failing due to a lint failure.

@edugfilho edugfilho merged commit 9400bf7 into main Sep 22, 2025
7 checks passed
@edugfilho edugfilho deleted the deng-9708 branch September 22, 2025 17:16
@edugfilho edugfilho changed the title DENG-9703: STMO links lead to appropriate templates that replace copy/paste of query DENG-9703: STMO links lead to appropriate query templates that are populated Sep 23, 2025
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.

3 participants