Update etherpad to 1.8.12 #10

Closed
opened 2021-02-25 20:06:57 +00:00 by meaz · 7 comments
Owner
  • Test updating on vagrant with vanilla
  • Test updating on vagrant with host_vars
  • Update the role to uninstall plugins if needed
  • PR for role merged
  • Set ep_adminpads to false in host_vars
  • Bump version in host_vars
  • PR for host_vars merged
- [x] Test updating on vagrant with vanilla - [x] Test updating on vagrant with host_vars - [x] Update the role to uninstall plugins if needed - [x] PR for role merged - [x] Set ep_adminpads to false in host_vars - [x] Bump version in host_vars - [x] PR for host_vars merged
Author
Owner

Updating from 1.8.7 to a higher version breaks because of ep_adminpads. It is not maintained anymore so should be replaced by ep_adminpads2 as explained here.

Once removed, it kind of work. I'm updating the role to have plugins set to false to be uninstalled.

But there is still an issue, as the content of pad doest't load..

Updating from 1.8.7 to a higher version breaks because of `ep_adminpads`. It is not maintained anymore so should be replaced by `ep_adminpads2` as explained [here](https://github.com/ether/ep_adminpads2). Once removed, it kind of work. I'm updating the role to have plugins set to false to be uninstalled. But there is still an issue, as the content of pad doest't load..
meaz changed title from Update etherpad to 1.8.10 to Update etherpad to 1.8.11 2021-03-02 08:07:53 +00:00
muppeth added this to the (deleted) project 2021-03-02 11:01:01 +00:00
Author
Owner

Role updated, PR pushed.
Everything works on my side, but need a double check ;)

Role updated, PR pushed. Everything works on my side, but need a double check ;)
meaz changed title from Update etherpad to 1.8.11 to Update etherpad to 1.8.12 2021-03-05 11:14:54 +00:00
Author
Owner

Oh! It is already in 1.8.12 now! Just tested it, it works great.

Oh! It is already in 1.8.12 now! Just tested it, it works great.
Owner

There is a feature request from a user to add: ep_markdown https://www.npmjs.com/package/ep_markdown

I think, if it works and there is no reason to not have it (was there issue with export? perhaps @meaz), we could add it to the role defaults.

I will test it later today.

There is a feature request from a user to add: ep_markdown https://www.npmjs.com/package/ep_markdown I think, if it works and there is no reason to not have it (was there issue with export? perhaps @meaz), we could add it to the role defaults. I will test it later today.
Author
Owner

It was removed because we had issue with it.

I've tested it, it works fine, so it can be added.

As we're on that, I like the ep_author_hover one. It is working fine so we could add it if you agree.

It was removed because we had issue with it. I've tested it, it works fine, so it can be added. As we're on that, I like the [ep_author_hover](https://github.com/ether/ep_author_hover) one. It is working fine so we could add it if you agree.
muppeth added this to the 16.03.2021 - #17 milestone 2021-03-16 07:42:36 +00:00
Owner

Great. Canyou adjust host_vars and add those two to defaults. I think they are nice to have.

Great. Canyou adjust host_vars and add those two to defaults. I think they are nice to have.
Author
Owner

It is done.

It is done.
Sign in to join this conversation.
No Milestone
No project
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: Disroot/CHANGELOG#10
No description provided.