Skip to content

Spreadsheets#1189

Open
vitorpamplona wants to merge 11 commits intonostr-protocol:masterfrom
vitorpamplona:spreadsheets
Open

Spreadsheets#1189
vitorpamplona wants to merge 11 commits intonostr-protocol:masterfrom
vitorpamplona:spreadsheets

Conversation

@vitorpamplona
Copy link
Copy Markdown
Collaborator

@vitorpamplona vitorpamplona commented Apr 22, 2024

[ "d", "SheetStr Demo" ],
[ "data", "Sheet1", "J", "25", "3" ],
[ "data", "Sheet1", "J", "26", "5" ],
[ "data", "Sheet1", "J", "27", "=SUM(J25:J26)" ]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

are these formula's standardized? also what about custom formulas, are they out of scope for the nip?

@abh3po abh3po mentioned this pull request May 9, 2024
@arthurfranca
Copy link
Copy Markdown
Contributor

It may be a good idea to choose a list to hold all sheet references the user is interested in, because one can't rely on authors filter and anyone can add the user to a p tag on a random sheet.

@vitorpamplona
Copy link
Copy Markdown
Collaborator Author

It may be a good idea to choose a list to hold all sheet references the user is interested in

Yep, I was thinking about an unbound list that just points to all "accepted" spreadsheets.

Comment on lines +19 to +20
["data", "<sheet name>", "<column letter>", "<row number>", "<value>"],
["data", "<sheet name>", "<column letter>", "<row number>", "<value>", "<style>"],
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

instead of 3 tags and requiring the decoding of column letters => column number we could add a ["sheet", "Sheet Name"] tag, and then do 0:0:1 to refer to sheet tag 0, and the cell at position row 1 (y), column 0 (x)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

The issue is how to not break the map between these names and the formulas, since formulas use the names instead of indexes... If one client wants to start with A while others want to start with a letter of their alphabet, they wouldn't be able to translate indexes without mapping back to A first and then translating the formula.

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.

5 participants