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

Bugfix: creating new doc type properties sort order doesn't count existing properties - puts it at top (+1) #2523

Merged

Conversation

cheeseytoastie
Copy link
Contributor

@cheeseytoastie cheeseytoastie commented Nov 7, 2024

Description

Fixes umbraco/Umbraco-CMS#17457

Creating new doc type properties sort order doesn't count existing properties.

Looking at this it seems that the hardwork is done to workout the sort order and assign it to preset but it's not returned undefined was (probably as a dev placeholder that's gone unnoticed!).

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

How to test?

Checklist

  • If my change requires a change to the documentation, I have updated the documentation in this pull request.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.

umbraco/Umbraco-CMS#17457

Creating new doc type properties sort order doesn't count existing properties.

Looking at this it seems that the hardwork is done to workout the sort order and assign it to preset but it's not returned undefined was (probably as a dev placeholder that's gone unnoticed!).
Copy link
Contributor

@JasonElkin JasonElkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested. Fixes the problem as described.

Thanks @cheeseytoastie!

@leekelleher leekelleher changed the title Bug 17457 - creating new doc type properties sort order doesn't count existing properties - puts it at top (+1) Bugfix: creating new doc type properties sort order doesn't count existing properties - puts it at top (+1) Nov 7, 2024
@leekelleher leekelleher self-requested a review November 7, 2024 17:56
@leekelleher leekelleher enabled auto-merge (squash) November 7, 2024 18:01
Copy link
Member

@leekelleher leekelleher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cheeseytoastie Thank you for the fix! 🚀
Thanks to @JasonElkin for testing, I've also verified the fix. Go team! 🙌

@leekelleher leekelleher merged commit 554ce0e into umbraco:main Nov 7, 2024
9 checks passed
@umbrabot
Copy link

umbrabot commented Dec 4, 2024

Hi there @cheeseytoastie!

First of all: A big #H5YR for making an Umbraco related contribution during Hacktoberfest! We are very thankful for the huge amount of PRs submitted, and all the amazing work you've been doing 🥇

Due to the amazing work you and others in the community have been doing, we've had a bit of a hard time keeping up. 😅 While all of the PRs for Hacktoberfest might not have been merged yet, you still qualify for receiving some Umbraco swag, congratulations! 🎉

In the spirit of Hacktoberfest we've prepared some exclusive Umbraco swag for all our contributors - including you!
This year's swag is a custom designed notebook and custom Umbraco Hacktoberfest sticker:

image

As an alternative choice, you can opt-out of receiving anything and ask us to help improve the planet instead by planting a tree on your behalf. 🌳

Receive your swag or plant a tree! 👈 Please follow this link to fill out and submit the form, before December 25th, 2024, 23:59:00 UTC.

Following this date we'll be sending out all the swag, but please note that it might not reach your doorstep for a few weeks/months, so please bear with us and be patient 🙏

The only thing left to say is thank you so much for participating in Hacktoberfest! We really appreciate the help!

Kind regards,
The various Umbraco teams.

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

Successfully merging this pull request may close these issues.

15rc: Creating new doc type properties sort order doesn't count existing properties
5 participants