Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add versioning to custom html modules #11136

Closed
photodude opened this issue Jul 14, 2016 · 9 comments
Closed

Add versioning to custom html modules #11136

photodude opened this issue Jul 14, 2016 · 9 comments

Comments

@photodude
Copy link
Contributor

photodude commented Jul 14, 2016

Steps to reproduce the issue

with content versioning on,

  1. create a custom HTML module (save/close)
  2. edit custom HTML module (save close)

the pervious version should be maintained in the content history, but no version history is maintained

Expected result

Version history should by maintained for custom HTML modules

Actual result

no version history is maintained

Additional comments

custom HTML modules are very much like articles, and some site owners use custom HTML modules like micro content used through out a site. These same site owners may either desire to keep a content history or are required to keep such a history for corporate or regalatory reasons.

@brianteeman
Copy link
Contributor

I dont remember why but this request was denied previously #7019 (comment)


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11136.

@photodude
Copy link
Contributor Author

photodude commented Jul 15, 2016

@brianteeman could the reason be somewhere on Gilp? (I'm not currently on Glip)

If we can't find a Justification for why this request was previously denied, I suggest we move it to a RFC.

@brianteeman
Copy link
Contributor

On 15 Jul 2016 8:53 p.m., "Walt Sorensen" notifications@github.com wrote:

@brianteeman could the reason be somewhere on Gilp?

Unfortunately not it was a decision made at the cleanup sprint

@chrisdavenport
Copy link
Contributor

@photodude I don't recall the reason either (and I was there) but I doubt it was a final "no way, ever" kind of thing, so if you would like to think through the technical and architectural issues and produce an RFC it will certainly be considered.

@mbabker
Copy link
Contributor

mbabker commented Jul 15, 2016

For one you'd basically have to version all modules or hack the UCM data
structure in a way that it can work with non-component type data. That's a
pretty big wall to get over when dealing with it.

On Friday, July 15, 2016, Chris Davenport notifications@github.com wrote:

@photodude https://github.com/photodude I don't recall the reason
either (and I was there) but I doubt it was a final "no way, ever" kind of
thing, so if you would like to think through the technical and
architectural issues and produce an RFC it will certainly be considered.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#11136 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWfoSB_TKHXRSltu5NbLJwWd77QsbL2ks5qV-_igaJpZM4JM6XC
.

@photodude
Copy link
Contributor Author

Thanks @chrisdavenport, I'll consider writing up some technical and architectural solutions.

@mbabker I agree that UCM structure is a big wall to deal with. IMO Modules should be versionable by type, and only if the module is set up to support versioning.

For the time being my efforts are going to be focused on the PHPCS 2.x Coding Standards and possibly the HHVM testing for some time. I'll definitely come back to this at some point but it's lower on my consideration list.

@brianteeman
Copy link
Contributor

I remember now that the reasoning was to do with the issues with UCM and with the roadmap for page builder which would possibly make this need redundant


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11136.

@photodude
Copy link
Contributor Author

page builder is on the Joomla 4 roadmap right?

I still think this maybe worth pursuing, pending the implementation of page builder.

@brianteeman
Copy link
Contributor

If you want to go ahead and write some code then fine I'm certainly not going to stop you. I am going to close this now and if/when you create some code that will be a new PR


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11136.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants