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: enable notebooks and annotations #21972

Merged
merged 5 commits into from
Jul 29, 2021

Conversation

williamhbaker
Copy link
Contributor

Closes #21217
Closes #21266

This PR enables notebooks and annotations by removing the feature flag for notebooks and notebooksApi, both of which are no longer checked for in the UI (they are always on in the latest UI), and turns the feature flag for annotations to be true by default.

I also cleaned up a few other UI-related feature flags that have been removed from the UI

@@ -94,22 +74,6 @@
default: false
expose: true

- name: UI CSV Uploader
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed from the UI in influxdata/ui#1130

expose: true
lifetime: temporary

- name: Editable Telegraf Configurations
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed from the UI in influxdata/ui#1597

# not if the notebooks button is enabled on the sidebar. The notebooks flag (above) controls
# the display of the notebooks button on the sidebar. The notebooksAPI flag should be set to true
# to ensure the UI sends requests that the OSS backend can handle.
- name: Notebooks Service API
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed from the UI in influxdata/ui#1809

@@ -55,26 +55,6 @@
contact: Query Team
lifetime: permanent

- name: Notebooks
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed from the UI in influxdata/ui#1995

Copy link
Contributor

@davidby-influx davidby-influx left a comment

Choose a reason for hiding this comment

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

Seems good.

@williamhbaker williamhbaker merged commit 11e5ce0 into master Jul 29, 2021
@williamhbaker williamhbaker deleted the wb-notebooks-annotations-feat-flags branch July 29, 2021 22:32
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.

Remove feature flag for notebooks from OSS Remove feature flag for annotations from OSS
2 participants