Skip to content

added goToRevisionEvent hook, regexp support for pre and post values in ep.json - WONT MERGE#3408

Closed
ilmartyrk wants to merge 46 commits intoether:developfrom
citizenos:develop
Closed

added goToRevisionEvent hook, regexp support for pre and post values in ep.json - WONT MERGE#3408
ilmartyrk wants to merge 46 commits intoether:developfrom
citizenos:develop

Conversation

@ilmartyrk
Copy link
Contributor

@ilmartyrk ilmartyrk commented Jun 5, 2018

Added this hook to run custom functions in timeline view. For example currently ep_resize plugin does not work in that view and some of the content is displayed outside of the frame. That happens when during the editing process text has been longer than it is in the final version. Fixed htmlExport test: #3425

@muxator
Copy link
Contributor

muxator commented Aug 7, 2018

I would prefer to have PRs submitted as a linear series (== without merges and sync commits) on top of current develop.

I understand this is a new requirement, still without an explicit guideline.

The reason for which I am going to ask this is that bisecting a bug when the history is full of spurious commits becomes extremely slow, and impairs maintainability in the long run.

@muxator
Copy link
Contributor

muxator commented Aug 15, 2018

As promised, here is some documentation for pull requests.

Drop a line if not clear / need help.

@muxator muxator added this to the 1.8 milestone Aug 15, 2018
@ilmartyrk
Copy link
Contributor Author

@muxator, thank you

@muxator
Copy link
Contributor

muxator commented Dec 1, 2018

If we want to merge this PR, it should be rebased on current revision.

@muxator muxator modified the milestones: 1.7.5, 1.8 Feb 5, 2019
@muxator muxator modified the milestones: 1.8.0, 1.8.1 Dec 7, 2019
@JohnMcLear JohnMcLear changed the title added goToRevisionEvent hook, regexp support for pre and post values in ep.json added goToRevisionEvent hook, regexp support for pre and post values in ep.json - WONT MERGE Mar 23, 2020
@JohnMcLear
Copy link
Member

Note that the plugin work can't be merged due to conflicts with the async changes.

I will try pick out what I can.

@JohnMcLear
Copy link
Member

Both of these contributions are now in separate PRs so this can be closed.

@JohnMcLear JohnMcLear closed this Mar 30, 2020
@muxator muxator removed this from the 1.8.3 milestone Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants