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(templates): Remove templates #19300

Merged
merged 12 commits into from
Aug 25, 2020
Merged

Conversation

ebb-tide
Copy link
Contributor

@ebb-tide ebb-tide commented Aug 12, 2020

Closes #7609

Removed places in the UI where we enable imports and exports from 'old' templates.
Kept code for creating dashboards and variables from 'old' templates, because the UI still uses 'old' templates to create dashboards for telegraf configs and for demodata.
kept the @influxdb/influxdb-templates dependency because the UI still uses them to create dashboards for telegraf configs.

@ebb-tide ebb-tide force-pushed the deniz-remove-templates branch from ec9fd3b to ccb2c94 Compare August 12, 2020 16:53
@ebb-tide ebb-tide changed the title Deniz remove templates feat(templates): remove templates Aug 12, 2020
@ebb-tide ebb-tide changed the title feat(templates): remove templates feat(templates): Remove templates Aug 12, 2020
@ebb-tide ebb-tide force-pushed the deniz-remove-templates branch 2 times, most recently from 274acaf to 75ad7ef Compare August 12, 2020 18:54
Copy link
Contributor

@zoesteinkamp zoesteinkamp left a comment

Choose a reason for hiding this comment

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

Looks good to me!

Copy link
Contributor

@hoorayimhelping hoorayimhelping left a comment

Choose a reason for hiding this comment

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

great work, very thorough. are those js files at the top of the file list generated? the spacing in them is odd

// }

async getCreateDashboardItem(item){
async getCreateDashboardItem(item){
return await this.driver.findElement(By.css(`[data-testid^=add-resource-dropdown--][id='${item}']`));
}

Copy link
Contributor

Choose a reason for hiding this comment

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

what's up with the spacing in this file?

@ebb-tide ebb-tide force-pushed the deniz-remove-templates branch 3 times, most recently from dae9342 to 6c988a8 Compare August 18, 2020 04:34
@hoorayimhelping hoorayimhelping merged commit 543bbd1 into master Aug 25, 2020
@hoorayimhelping hoorayimhelping deleted the deniz-remove-templates branch August 25, 2020 15:47
hoorayimhelping added a commit that referenced this pull request Aug 26, 2020
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.

InfluxDB Documentation link broken for influxdb-data
3 participants