enh/import from deck

[juliusknorr]

• fixes Trello JSON Import Error #4818
• Refactor import framework to support multiple board impoards
• Basic implementation of importing deck json data
  • Boards
  • Stacks
  • Cards
  • Labels
  • Label assignments
  • User/group/circle assignments
• Tests
  • Import from json
  • Re-import previously exported json
• Documentation
  • Review documentation about import/export
  • limitations of import/export (attachments, comments, activity)
  • Document occ deck:import changes in changelog

Possible follow ups

• UI for importing since the API is already there
• Properly handle import issues and not fail but collect issues and still continue with the import

Notes

• PHP 8.0 is fine since we need it for 26

• Useful snipped for comparing the json export of a imported board

diff --git a/tests/integration/import/ImportExportTest.php b/tests/integration/import/ImportExportTest.php
index 00df316f..e3f75f3c 100644
--- a/tests/integration/import/ImportExportTest.php
+++ b/tests/integration/import/ImportExportTest.php
@@ -147,6 +147,8 @@ class ImportExportTest extends \Test\TestCase {
                $tmpExportFile = tempnam('/tmp', 'export');
                file_put_contents($tmpExportFile, $jsonOutput);

+               self::assertEquals(file_get_contents(__DIR__ . '/../../data/deck.json'), $jsonOutput);
+
                // cleanup test database
                $this->connection->rollBack();
                $this->connection->beginTransaction();

[cypress]

Passing run #1261 ↗︎

0

15

0

0

0

Details:

Merge 20f0e8e into 863e729...
Project: Deck	Commit: f8156e19b2 ℹ️
Status: Passed	Duration: 01:48 💡
Started: Aug 11, 2023 4:53 PM	Ended: Aug 11, 2023 4:55 PM

_{This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.}

[max-nextcloud]

Looks good thus far. Added some minor comments that came to my mind.

Since this is very php heavy might be good to get a review from a more php savvy person.

[max-nextcloud]

Comments are not exported, right? At least it says so in the export section - so maybe this should move into the section for the sources that come with comments?

[max-nextcloud]

I think it would be good to add some more info about the config file here. At least from the docs right now i would not know what to do.

[juliusknorr]

/backport to stable27

[juliusknorr]

/backport to stable27

[backportbot-nextcloud]

The backport to stable27 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable27
git pull origin stable27

# Create the new backport branch
git checkout -b fix/foo-stable27

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts. Resolve them.
git cherry-pick abc123

# Push the cherry pick commit to the remote repository and open a pull request
git push origin fix/foo-stable27

More info at https://docs.nextcloud.com/server/latest/developer_manual/getting_started/development_process.html#manual-backport