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

Introduce Cell Notes & Note Cells #4257

Merged
merged 33 commits into from
Aug 23, 2018
Merged

Introduce Cell Notes & Note Cells #4257

merged 33 commits into from
Aug 23, 2018

Conversation

alexpaxton
Copy link
Contributor

@alexpaxton alexpaxton commented Aug 21, 2018

Closes #4232
Closes #2445

moar-markdown-cell

Briefly describe your proposed changes:
Users can add an optional note to a cell via the CEO Visualization tab. They can specify whether to show the note when the cell's queries return no data or leave as a tooltip.

Users can set a cell to type Note which simply displays the contents of the note in the cell. Notes support markdown

  • CHANGELOG.md updated with a link to the PR (not the Issue)
  • Rebased/mergeable
  • Tests pass
  • swagger.json updated (if modified Go structs or API)

jaredscheib and others added 27 commits August 16, 2018 14:57
Requires further input to make sure this is effective at its
purpose.
Avoid overlap with associated cell whenever possible
Copy link
Contributor

@desa desa left a comment

Choose a reason for hiding this comment

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

@jaredscheib will this have the html sanitizing stuff in it, or will that be another PR?

@alexpaxton alexpaxton merged commit fc9cd71 into master Aug 23, 2018
@alexpaxton alexpaxton deleted the feat/cell-notes branch August 23, 2018 21:41
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