Skip to content

spreadsheet#5

Open
Siddhu-sathyan wants to merge 2 commits intoonesteinbv:16.0from
Siddhu-sathyan:spreadsheet
Open

spreadsheet#5
Siddhu-sathyan wants to merge 2 commits intoonesteinbv:16.0from
Siddhu-sathyan:spreadsheet

Conversation

@Siddhu-sathyan
Copy link

This PR contains the remaining documentation for the module Spreadsheets.

PS - this is the cleaned up PR after resolving the conflicts of the PR created in the previous repo - onesteinbv/docs#25

@Siddhu-sathyan Siddhu-sathyan force-pushed the spreadsheet branch 4 times, most recently from 8cf2a6b to 8c8910b Compare June 9, 2025 11:52
index.rst Outdated
docs/chat/chat
docs/dashboards/dashboards
docs/spreadsheets/index
docs/marketing/emailmarketing
Copy link
Member

Choose a reason for hiding this comment

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

Why do you remove these?

Copy link
Author

Choose a reason for hiding this comment

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

This link is removed from the index since a new path is created for email marketing -> docs/emailmarketing/

Copy link
Member

@tarteo tarteo Jun 24, 2025

Choose a reason for hiding this comment

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

Okay, here right? #4 For next time it's best to keep PRs self-contained

Copy link
Member

Choose a reason for hiding this comment

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

It can result in conflicts, like right now

@tarteo
Copy link
Member

tarteo commented Jun 10, 2025

@Siddhu-sathyan Thanks, great work keep it up! I took the liberty to fix the format of the toctree here 25cbb4a just needed some extra spaces :)

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.

2 participants