deck
deck copied to clipboard
Update TrelloJsonService.php
Fixes the following issue: https://github.com/nextcloud/deck/issues/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
- Resolves: https://github.com/nextcloud/deck/issues/5136
- Target version: main
Summary
TODO
- [ ] ...
Checklist
- [x] Code is properly formatted
- [x] Sign-off message is added to all commits
- [x] Tests (unit, integration, api and/or acceptance) are included
- [x] Documentation (manuals or wiki) has been updated or is not required
Can confirm this fixes the Problem also for importing with the TrelloApi System. Thanks!
/backport to stable29
/backport to stable28
/backport to stable27
Can confirm this fixes the Problem also for importing with the TrelloApi System. Thanks!
I've just successfully imported a Trello-Board using TrelloApi after applying the proposed changes. Works fine. :-)
Hello there, Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.
We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.
Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6
Thank you for contributing to Nextcloud and we hope to hear from you soon!
Would it be possible to get an official release soon to make it more easier for the people currently migrating from Trello to Nextcloud Deck following the monetisation of Trello?
@mulles Integration tests failed. Could you please rebase your branch?