Skip to content

Conversation

lauraneto
Copy link
Contributor

@lauraneto lauraneto commented Aug 14, 2025

lauraneto added 30 commits July 22, 2025 15:36
…. Display UTC offset instead of short offset name in label.
@lauraneto lauraneto requested a review from Copilot September 22, 2025 15:18
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 83 out of 84 changed files in this pull request and generated 4 comments.

Files not reviewed (1)
  • src/Umbraco.Web.UI.Client/package-lock.json: Language not supported

Copy link
Contributor

@AndyButland AndyButland left a comment

Choose a reason for hiding this comment

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

This is so good @lauraneto - really great piece of work.

I had no further comments - just pushed up a few additional XML header comments to complete the documentation on the new classes you've introduced.

Did some testing of all different types, and looks to work flawlessly to me:

image image

And from delivery API output:

        "dateOnly": "2025-09-23",
        "timeOnly": "12:14:06",
        "dateUnspecified": "2025-09-23T12:14:00",
        "dateLocal": "2025-09-23T12:14:00+02:00",
        "dateUtc": "2025-09-23T12:14:00+00:00",
        "dateWithTimezones": "2025-09-23T12:14:00+10:00"

@lauraneto lauraneto changed the title Property Editors: New Date Time picker with time zone support Property Editors: New Date Time property editors Sep 25, 2025
…operty-editor

# Conflicts:
#	src/Umbraco.Web.UI.Client/src/packages/core/validation/mixins/form-control.mixin.ts
…operty-editor

# Conflicts:
#	src/Umbraco.Web.UI.Client/package-lock.json
Copy link
Member

@nielslyngsoe nielslyngsoe left a comment

Choose a reason for hiding this comment

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

Looks good

@lauraneto lauraneto enabled auto-merge (squash) September 30, 2025 12:49
@lauraneto lauraneto merged commit 51575e5 into v17/dev Sep 30, 2025
39 of 42 checks passed
@lauraneto lauraneto deleted the v17/feature/new-datetime-with-timezone-property-editor branch September 30, 2025 13:21
leekelleher added a commit that referenced this pull request Oct 1, 2025
iOvergaard added a commit that referenced this pull request Oct 2, 2025
* Upgrade to Tiptap v3

* Uses `@ts-expect-error` to ignore the TS complication errors

These can be removed once Tiptap has resolved the TypeScript definitions.

* Off-topic: corrected `flags` property in the mock data

Added in PR #19915

* Update src/Umbraco.Web.UI.Client/src/packages/tiptap/extensions/link/link.tiptap-extension.ts

Co-authored-by: Copilot <[email protected]>

---------

Co-authored-by: Copilot <[email protected]>
Co-authored-by: Jacob Overgaard <[email protected]>
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.

3 participants