Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable27] Update TrelloJsonService.php #6643

Merged
merged 2 commits into from
Jan 2, 2025
Merged

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Jan 2, 2025

Backport of PR #5859

mulles added 2 commits January 2, 2025 14:58
Fixes the following issue: #5136
By fixing the following error when running 'php occ deck:import --system=TrelloJson'

Error: Call to a member function getUID() on string in /home/sites/site100028002/web/nextcloud.enklaave.org/apps/deck/lib/Service/Importer/Systems/TrelloJsonService.php:136

and 

Error: Call to a member function getUID() on string in /home/sites/site100028002/web/nextcloud.enklaave.org/apps/deck/lib/Service/Importer/Systems/TrelloJsonService.php:364


Signed-off-by: mulles <[email protected]>
removed trailing white space, I am sorry for that thx for your patience.

Signed-off-by: mulles <[email protected]>
Copy link
Contributor

github-actions bot commented Jan 2, 2025

🐢 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 67615 was 67430 (+0.27%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

Copy link

cypress bot commented Jan 2, 2025

Deck    Run #1960

Run Properties:  status check passed Passed #1960  •  git commit f403fa5a38 ℹ️: Merge 1bbc19f9e0ffeee1b6d55d4aaedf0f5e44195318 into 49996522e33a2030dcc6ec64525f...
Project Deck
Branch Review backport/5859/stable27
Run status status check passed Passed #1960
Run duration 03m 20s
Commit git commit f403fa5a38 ℹ️: Merge 1bbc19f9e0ffeee1b6d55d4aaedf0f5e44195318 into 49996522e33a2030dcc6ec64525f...
Committer backportbot[bot]
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 17
View all changes introduced in this branch ↗︎

@juliusknorr juliusknorr merged commit 89bb483 into stable27 Jan 2, 2025
22 checks passed
@juliusknorr juliusknorr deleted the backport/5859/stable27 branch January 2, 2025 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants