Skip to content

Address issue to release Version 3.6#330

Merged
benbalter merged 13 commits intowp-document-revisions:mainfrom
NeilWJames:main
Dec 6, 2023
Merged

Address issue to release Version 3.6#330
benbalter merged 13 commits intowp-document-revisions:mainfrom
NeilWJames:main

Conversation

@NeilWJames
Copy link
Copy Markdown
Collaborator

@benbalter
Sorry it has taken a time to revert with the fix to the problem I had raised with you to release V3.6.

I had hoped to address the issues for WPML.

Whilst I think think that I have them sorted (mentally at least), I think the most appropriate way to deal with it is to a plugin to go in the Cookbook that will be the functions to interface WPML, and to add filters and/or actions (if necessary) to the plugin.

I have changed the way the PDF image denotes that it has hidden itself to be better aligned with WPML processing. It will be marginally more efficient in processing than having a separate piece of metadata.

I have also fixed the issue raised today on Github which is a text typo.

Separately I wrote a document giving information on the data structures used. This was if I needed to go the the WPML folks to define how the plugin works. It certainly can be better written, but I thought it is better to be out there rather than simply on my PC.

@NeilWJames NeilWJames requested a review from benbalter November 30, 2023 16:00
@codecov-commenter

This comment was marked as outdated.

@benbalter benbalter merged commit e0fff4b into wp-document-revisions:main Dec 6, 2023
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.

3 participants