coop-cloud/mediawiki
coop-cloud
/
mediawiki
6
0
Fork 3
Code Issues 6 Pull Requests Releases Wiki Activity

remove parsoid image #32

Closed
iexos wants to merge 1 commits from iexos:rm-parsoid into main
pull from: iexos:rm-parsoid
merge into: coop-cloud:main
Conversation 1 Commits 1 Files Changed 2 -25
iexos commented 2023-05-26 15:33:13 +00:00
Member
Copy Link

The recipe breaks for me after the first edit with Visual editor, causing parsoid to error out every time. After removing parsoid, it works fine.

Explanation: parsoid/php is now integrated with the core image, configuring the outdated parsoid/js causes errors

See also:

The recipe breaks for me after the first edit with Visual editor, causing parsoid to error out every time. After removing parsoid, it works fine. Explanation: parsoid/php is now integrated with the core image, configuring the outdated parsoid/js causes errors See also: - https://www.mediawiki.org/wiki/Parsoid#Installation - https://github.com/thenets/docker-parsoid/issues/16#issuecomment-813402335 - breakage probably caused by this release: https://www.mediawiki.org/wiki/Parsoid/Releases#0.15.1_.28released_Sep_14.2C_2022.29
iexos added 1 commit 2023-05-26 15:33:20 +00:00
continuous-integration/drone/pr Build is failing Details
e37bf94205 remove parsoid image 
parsoid/php is now integrated with the core image, configuring the outdated parsoid/js causes errors
https://www.mediawiki.org/wiki/Parsoid#Installation
decentral1se commented 2023-05-28 14:10:29 +00:00
Owner
Copy Link

Tysm @iexos for this change! It's been tested and deployed already via https://git.coopcloud.tech/coop-cloud/mediawiki/pulls/31/files and @flancian says it's working 💯 Since I don't want to cause rebase issues with PRs, I'm gonna close this one and then review #31 which should get merged. Will add accreditation on the PR commit!

Tysm @iexos for this change! It's been tested and deployed already via https://git.coopcloud.tech/coop-cloud/mediawiki/pulls/31/files and @flancian says it's working 💯 Since I don't want to cause rebase issues with PRs, I'm gonna close this one and then review `#31` which should get merged. Will add accreditation on the PR commit!
👍 1
decentral1se closed this pull request 2023-05-28 14:10:33 +00:00
iexos deleted branch rm-parsoid 2023-05-28 14:34:52 +00:00
Some checks failed
continuous-integration/drone/pr Build is failing
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: coop-cloud/mediawiki#32
No description provided.
Delete Branch "iexos:rm-parsoid"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?