Skip to content

[rel-db] Fix meeting.import#3281

Closed
luisa-beerboom wants to merge 112 commits intoOpenSlides:feature/relational-dbfrom
luisa-beerboom:rel-db-fix-meeting-import
Closed

[rel-db] Fix meeting.import#3281
luisa-beerboom wants to merge 112 commits intoOpenSlides:feature/relational-dbfrom
luisa-beerboom:rel-db-fix-meeting-import

Conversation

@luisa-beerboom
Copy link
Copy Markdown
Member

@luisa-beerboom luisa-beerboom commented Jan 20, 2026

Currently based on #3152

I fixed the content of the failing meeting.import test by using the export_meeting function from export_helper.py. I then reinstated the presenter code because it is still the desired behaviour.

Should the problem not be fixable another way, it may be possible to get the test to run by replacing the presenter call code with export = export_meeting(self.datastore, 1, datetime_decimal_to_string=True)

hjanott and others added 6 commits January 18, 2026 00:04
 * requested changes
 * fix setting to running as first action
 * test fixes
 * docs
 * replace trigger
 * hard coded ORIGIN_COLLECTIONS
 * unskip initial import tests
 * update meta
Co-authored-by: luisa-beerboom <101706784+luisa-beerboom@users.noreply.github.com>
Co-authored-by: peb-adr <adrichter97@gmail.com>
Co-authored-by: luisa-beerboom <101706784+luisa-beerboom@users.noreply.github.com>
@luisa-beerboom luisa-beerboom added this to the 4.3 milestone Jan 20, 2026
@luisa-beerboom luisa-beerboom self-assigned this Jan 20, 2026
@luisa-beerboom luisa-beerboom added the enhancement General enhancement which is neither bug nor feature label Jan 20, 2026
@luisa-beerboom
Copy link
Copy Markdown
Member Author

luisa-beerboom commented Jan 21, 2026

Currently there is one test in meeting.import that doesn't work because of auth problems.
Unfortunately that test is important, so we'll have to see how to fix these auth things.

@luisa-beerboom luisa-beerboom force-pushed the rel-db-fix-meeting-import branch from 6c7ee9f to ed5d353 Compare January 21, 2026 13:05
@luisa-beerboom luisa-beerboom removed their assignment Jan 23, 2026
@luisa-beerboom luisa-beerboom marked this pull request as ready for review January 23, 2026 09:26
@luisa-beerboom
Copy link
Copy Markdown
Member Author

Replaced by #3287

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement General enhancement which is neither bug nor feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants