Date: 03/10/2024
All minutes published on: https://forum.collaboraonline.com/tag/meeting-minutes
Attendees (write your name):
- Pedro, Bayram, Darshan, Gulsah, Skyler, Anna, Attila, Caolan, Joao, Thorsten, Hub, Gabriel, Sven, Gokay, Timur
Testing
-
Files – ( password is coolmeeting)
-
Calc - https://staging-perf.eu.collaboraonline.com/nextcloud/index.php/s/BkG5k5THbjwZRKJ
-
Impress - https://staging-perf.eu.collaboraonline.com/nextcloud/index.php/s/5WBqk42nG5ziiNF
-
Writer - https://staging-perf.eu.collaboraonline.com/nextcloud/index.php/s/WrwXdbBkcn7xJbe
-
Features, user flows (Anyone can add here the things they need to test)
-
[Application] [Requester]: what to test
-
.
-
- Release Schedule
-
from master: CODE 24.04.8.1 “red” this week
-
CODE green 24.04.8.2 next week
-
Mobile
-
in the progress of making a release now – snapshot/betas are available for Android/iOS.
-
Fixing an iOS issue w/ version numbering before sending to QA.
-
Expecting public release to be ~Monday/Tuesday next week (although this may change if QA finds serious release blockers)
-
-
-
PRs in need for attention (Please add here any PR that needs attention)
-
PRs without reviewer https://github.com/orgs/CollaboraOnline/projects/3/views/2 now 1
-
Antranig Vartanian
-
Wants to contribute any FreeBSD work
-
Andras Timar: Another contributor was working on that → Gleb P. was the community maintainer
-
Antranig: it seems the image you were using was removed from Cirrus community
-
So we ended up disabling the Cirrus CI (FreeBSD) because the service was broken
-
I have talked with them and it seems they are using the Google cloud platform. But this means that every time we have a new version we need to wait for Google’s approval
-
Demo server is up and running
-
The goal for the end of the year would be
-
-
-
Sahil
-
LO Contributor from the last year
-
GSOC → UI customization
- But there is room for improvement, I feel that it should be much more flexible
-
Little experience with coding
-
Darshan:
-
Already has online build
-
Sahil claimed on of the easy hacks https://github.com/CollaboraOnline/online/issues/8566
-
https://github.com/CollaboraOnline/online/pull/10027 => Pedro needs to review this
-
-
-
Nicolas
-
.
-
Ankush
-
coder – working in finance industry
-
working on providing them with a tool here
-
started learning about COOL – two to three weeks back.
-
-
Some questions in the forum
-
Interested in improving finance bits …
- once setup – will be looking at features …
-
Forum discussion :
-
-
Gabriel
-
Busy with the conference https://omniopencon.org/
-
Some time for hacking and bring some new contributoes
-
Plan to do next year with more time to prepare
-
-
browser: add last modification and document status at the bottom #4531
-
https://github.com/CollaboraOnline/online/pull/4531 options where we can set that:
-
in review
-
Did the requested changes by Szymon but still just locally, will push later
-
-
Also now busy with https://omniopencon.org/ September 26-28, 2024
- Will have a talk on integrating COOL with 1&1
-
-
-
Working now on something related to CSV and probably will backport it but I’m waiting for some feedback → https://gerrit.libreoffice.org/c/core/+/165905
-
this was mostly useful for converting CSV → PDF – because separators were not found
-
idea was to detect automatically at conversion.
-
First patch – adds character-set and separator settings
-
can use these when you pass the filter parameters.
-
Instead of separator & character set put ‘detect’
-
-
Would be nice to use it in the import dialog.
-
-
Would love to help out with design for serializing settings (Michael)
-
Julius
-
Getting much more frequent races causing a download of the file … when clicked. (Michael)
-
seems like a richdocuments / Nextcloud race… perhaps more plugins installed …
-
now have draw.io/BPMN/etc. etc. in the list here … probably related to this.
-
Finally have a reproducer on our instance
-
Seems the viewer API has a design flaw in that regard
-
Our files engineering-team is looking into having a work around
-
-
Small bugfix release of richdocuments later (probably today)
-
branding customization fix for logo
-
iframe accessibility
-
Fix issue where files were locked during preview generation with network timeouts which could have lead to failed saves
- There is still a risk, but less likely, currently thinking about better ways to handle this
-
-
Nextcloud 30 upcoming, currently in beta
-
Integration with extract/transform document API
-
Create new documents with form from templates
-
-
Nextcloud Conference is happening Sept 14-15, contributor week afterwards
-
Nightly CODE docker image for easier testing
-
Maybe we should have --enable-experimental in here by default in the online configure line to match what I think we build CODE with by default (Caolán)
-
Thorsten
- [No update]
-
Attila
No update
-
Andras
-
Display debug info in About box (e.g. triple click to see them) https://github.com/CollaboraOnline/online/pull/10113 → merged
-
Sync default between coolwsd.xml and code. https://github.com/CollaboraOnline/online/pull/10129
-
-
Szymon
-
Fail cypress in CI on exceptions
- cypress: fail on exceptions even without debug #9927 https://github.com/CollaboraOnline/online/pull/9927
-
Fixed TypeErrors
-
Reverted the scroll window change to fix Function Wizard in release
-
Reviews and merging RED phase PRs
-
Next: Pedro’s report - now unotoolbuttons are being enable/disable but the inner child is not being enabled or disabled: search previous and search next
-
Darshan
-
PR:
-
Fix : Automatic focus jump during tab navigation
- need to replace hard coded, Found new approach to add a recursive method to find widget information from instance object in jsDialog https://github.com/CollaboraOnline/online/pull/10109
-
Feature: Save status notification #10144 => **Needs review
**https://github.com/CollaboraOnline/online/pull/10144 -
Fix keyboard shortcuts not working after row/column selection in Calc => Needs review https://github.com/CollaboraOnline/online/pull/10131
-
Cursor pointer not allowed for disabled buttons #10156 https://github.com/CollaboraOnline/online/pull/10156 => Merged
-
fix dark mode & invert BG icon should be highlighted on mode change #10081 https://github.com/CollaboraOnline/online/pull/10081 => Merged
-
-
-
Aron
- nothing to add
-
Caolan
-
Working on comment @mentions
-
Remove a synchronous DNS lookup and do the work earlier inexisting async DNS
- https://github.com/CollaboraOnline/online/pull/10138 (needs review)
-
Auto committed Watchdog profiles
-
Bayram
-
(In Progress) - Enhancement - Calc: Autofill: show the value that the latest cell will get once the mouse button is released
-
core: https://gerrit.libreoffice.org/c/core/+/172589 (Merged - 24.04 - master)
-
PR: https://github.com/CollaboraOnline/online/pull/10037 (WIP)
-
core: send row&column address (e.g. 1!R3C5) for correct location (thanks to Caolan for the idea)
-
then use _parseCellRange() and _cellRangeToTwipRect() for cellRange (this is also used by “comments”) (Caolan)
-
finally, auto-fill popup location is now correctly calculated. we can show the popup in the correct/desired place. (bottom-right of the cell) (with help from Gökay – thanks!)
-
now: trying to fix other small bugs → refactor → test with a spreadsheet that uses “view, freeze rows and columns” …
-
-
-
(Closed) - Display the name of the respective document more completely and prominently in the title line → was under review → tested & closed
-
PR: https://github.com/CollaboraOnline/online/pull/10098
- use available space for document name. make the title bold for more visibility.
-
-
Skyler
-
Core file formats work is merged! May need supporting patches from integrators (e.g. Nextcloud does not know about the new files yet) but generally these are openable in Collabora Online
-
Working on exporting impress presentations to different formats
-
Currently looking at flash – since as it’s a legacy file format that is still (unfortunately) in use
-
Support was removed from LibreOffice a while ago, so going back over that patch to see if I can back out sensibly
-
-
Mobile releases are in progress – there’s an Android snapshot and an iOS testflight. Currently fixing iOS version numbers before sending to QA for a look-over. Likely public release date is early next week (Monday, Tuesday)
-
Pranam
-
Calc formula tooltips now available in each users language (earlier it was based on language of the first user who loaded the document) → tested today, it works!
-
Various improvements for dark/light theme text in calc when font has automatic colors → let’s test this today
-
Fixed resolved comments would have highlighted on reload even if resolved comments are hidden.
-
-
Gökay
Continues:
-
Various bug fixes & reviews.
-
Grid in impress → Adding switch buttons and implementing snapping to grid functionality.
-
Impress notes view. Switch from notes to normal view
Next:
-
Copy text with comments issue. Has a PR but needs to check the tests also.
-
Hubert
- No updates.
-
Gulsah
-
Hyperlink preview glitch - https://github.com/CollaboraOnline/online/issues/9013
-
Sent a patch to gerrit: https://gerrit.libreoffice.org/c/core/+/173726
-
First merged then reverted because of the cypress failing. Investigating…
-
-
Better calc comment sign https://github.com/CollaboraOnline/online/issues/7130
-
Desktop version has the expected triangle sign, But online shows the old red rectangle, I’ve investigated the pointed patch but didn’t help the find solution. It seems red rectangle still comes from core, we don’t create it on online. I’m debugging to find where we create the sign.
-
Paused to work to solve problem above.
-
-
Vivek
-
Sven
-
Timur
-
Just observing.
-
Méven
- work in progress implementing curved lines for impress
-
Nick
-
Correctly receiving jsdialog online now, and have basic navigator working now
-
Need to:
-
Create open/close button
-
Fix navigator closing
-
Combine with slide sorter for impress
-
-
Back at university now so will have less time for collabora
-
Events
-
OmniOpenCon
-
September 26-28, 2024 in Bucharest
-
Accepting now talk submissions (can be also remotely) till the end of this week (37)
Forum Topics
-
Github:
-
New easy hack : perf: improve ScFlatUInt16RowSegments::dumpAsString() #10060 https://github.com/CollaboraOnline/online/issues/10060 => DONE
-
New contributor name Sopby (https://github.com/sopyb) doing very good. He already completed a easy hack related to performance improvement.
-
Now on to the next one : Calc: reduce protocol thrash … #8159
-
Got a message from contributor name Parth: waiting for his patches to be reviewed. https://github.com/CollaboraOnline/online/pulls/rparth07 => I have replied to his message and encourage to take a look in issues section for some more fun tickets and meanwhile we will review his patches
-
-
Top forum topics from last week:
-
Top topics from last week
-
This week forum seems to be quite don’t know what is the reason still we had few topics that we answered and closed as resolved => Thanks Miklos for helping with python related questions
-
Weblate status for September month has been created https://forum.collaboraonline.com/t/weblate-stats-september-2024/3028
-
502 Bad Gateway in NextCloud after dist-upgrade (–> Ubuntu 24.04) => Good to see this type of discussion in forum. Now user are not only just asking questions but also they are posting very useful solutions which they encountered and solved while installing COOL with NC etc…
-
-
Community page
-
I’m working on adding a new section to our community website, Collabora Online, to list all the contributors to the COOL repository
• Generate contributor list with metadata and HTML layout #10068 https://github.com/CollaboraOnline/online/pull/10068 => MERGED• Credits page for COOL contributors #128 => No need for this as we have a CRON job ready https://github.com/CollaboraOnline/CollaboraOnline.github.io/pull/128 -
CRON job : this GH workflow will run every day and update contributors list on out IO repository https://github.com/CollaboraOnline/online/pull/10126 => Review Needed
Next meeting
- 10/10/202