Skip to content

Conversation

nhatcher
Copy link
Member

@nhatcher nhatcher commented Jun 7, 2025

Hi @dg-ac , I am hesitant to merge this.

On the one hand it is cool, on the other I am not not sure how useful it is. Maybe we can have a common:

Export to:

  • PNG
  • Markup/down
  • ...

NB: You can't test this feature in the widget/storybook but will work on the app.

@nhatcher nhatcher requested a review from dg-ac June 7, 2025 16:18
@nhatcher nhatcher self-assigned this Jun 7, 2025
@nhatcher nhatcher force-pushed the feature/nicolas-markup branch from 36a275b to 6740a43 Compare June 8, 2025 10:41
@dg-ac
Copy link
Contributor

dg-ac commented Jun 13, 2025

@nhatcher I'll take a look this weekend.

@BrianHung
Copy link
Contributor

I wouldn't do a sheet to markdown method. I would do a getSheetValues or getSheetContent that takes rowStart,rowEnd,colStart,colEnd and returns a 2D array of string values.

Then the developer can decide whether to convert that into a CSV, TSV, HTML table, or a Markdown table.

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