leekelleher / umbraco-contentment

Contentment for Umbraco - a state of happiness and satisfaction
https://marketplace.umbraco.com/package/umbraco.community.contentment
Mozilla Public License 2.0
155 stars 70 forks source link

Added new ContentmentContentContext aware XPathParser #312

Closed busrasengul closed 1 year ago

busrasengul commented 1 year ago

Replace $parent with actual parent id.

Description

This PR is now changing the XPath $parent query to actual parent id. And modified where it was used.

We weren't sure where to put the class in subject, so we've put it under a new folder called Xml as the Umbraco one was under the Xml folder. We also couldn't be sure if it should be dependency injectable or not. Happy to change/modify anything.

Related Issues?

307

Types of changes

Checklist

leekelleher commented 1 year ago

Thank you @busrasengul. I'm on holiday at the moment, but will check this PR once I'm back. 👍

busrasengul commented 1 year ago

I've merged your changes on to my PR @leekelleher Thank you. Looks all great for me :)

callumbwhyte commented 1 year ago

Hey @leekelleher,

Just wondering if you had any plans to release v4.5.0 anytime soon? Would be awesome to have this fix into our project!

If there's bigger plans at play here (I can see some incomplete features on the milestone) then no worries / pressure!

Cheers, Callum

leekelleher commented 1 year ago

@callumbwhyte No bigger plans, just dropped off my radar - I haven't even opened Visual Studio in the past couple of weeks. 😆 I'll get a v4.4.3 patch release out soon (now that you've mentioned this). v4.5.0 (with Data Picker editor) isn't ready yet, (again fell completely off my radar). 😆

leekelleher commented 1 year ago

@callumbwhyte @busrasengul Contentment v4.4.3 has been released.