Skip to content

Conversation

@davidtranhp
Copy link

No description provided.

chrisandrewmann and others added 16 commits September 10, 2025 12:26
Fix bug where sort order in pivot causes crash due to changed function
name getGroupByDisplayLabel to getMeasureDisplayName.
Signed-off-by pedrobaeza
…odule offers

Reduce the logic of the controller by using the function that spreadsheet module offers and avoid error when rendering images on the xlsx file.
Signed-off-by pedrobaeza
Signed-off-by CarlosRoca13
…preadsheet

Before these changes, when a table was added to an existing spreadsheet, it was always being added to row 0 of the sheet, since the call to getCell always returned undefined.

After making these changes, the call is made correctly and the cell’s content is retrieved. Since the isEmpty function no longer exists, we now check directly whether it contains the content key.
Signed-off-by pedrobaeza
Signed-off-by pedrobaeza
@davidtranhp davidtranhp added this to the 17.0 milestone Jan 7, 2026
@viinbot viinbot added the 🚀 Queue PR/Commit is added in build queue label Jan 7, 2026
@viinbot viinbot added 🚀 Building PR/Commit is start building all step in script ⚠️ Failed PR/Commit is failed when run script and removed 🚀 Queue PR/Commit is added in build queue 🚀 Building PR/Commit is start building all step in script labels Jan 7, 2026
@viinbot
Copy link

viinbot commented Jan 7, 2026

@OCA-git-bot Viindoo Test Suite has failed!

@viinbot viinbot added 🚀 Queue PR/Commit is added in build queue 🚀 Building PR/Commit is start building all step in script 🚀 Running PR/Commit is done for everything and ready for manually test and removed ⚠️ Failed PR/Commit is failed when run script 🚀 Queue PR/Commit is added in build queue 🚀 Building PR/Commit is start building all step in script labels Jan 7, 2026
@viinbot
Copy link

viinbot commented Jan 7, 2026

@davidtranhp Viindoo Test Suite has passed!

@davidtranhp davidtranhp merged commit d3d3843 into Viindoo:17.0 Jan 8, 2026
5 checks passed
@davidtranhp davidtranhp deleted the merged_from_upstream17_260107_01 branch January 8, 2026 04:54
@viinbot viinbot added 🚩 Done PR/Commit is finished manually test and closed instance and removed 🚀 Running PR/Commit is done for everything and ready for manually test labels Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚩 Done PR/Commit is finished manually test and closed instance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants