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 upper/lower presentation format for am/pm in fromMillis #644

Merged
merged 1 commit into from
Aug 10, 2023

Conversation

alexwoodgate
Copy link

Fixes #574

Updates $fromMillis() function to support am/pm PN presentation format to output AM/PM and adds test cases for PN and Pn (existing test case for default presentation covers just P).

Note: I did not implement [PNn] for Am/Pm, it's not shown in the examples https://www.w3.org/TR/xpath-functions-31/#date-time-examples so I assumed it's not commonly used and did not implement it to keep the fix simple. Happy to revisit that if needed.

Signed-off-by: Alex Woodgate ajwoodgate@gmail.com

Signed-off-by: Alex Woodgate <ajwoodgate@gmail.com>
Copy link
Member

@mattbaileyuk mattbaileyuk left a comment

Choose a reason for hiding this comment

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

Thanks for contributing this fix. There's probably more iterative work on some of these picture notations, but good to get this one ticked off.

@mattbaileyuk mattbaileyuk merged commit 7a7f2b5 into jsonata-js:master Aug 10, 2023
6 checks passed
renovate bot referenced this pull request in solid-design-system/solid Mar 1, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [jsonata](http://jsonata.org/)
([source](https://togithub.com/jsonata-js/jsonata)) | [`2.0.3` ->
`2.0.4`](https://renovatebot.com/diffs/npm/jsonata/2.0.3/2.0.4) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/jsonata/2.0.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/jsonata/2.0.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/jsonata/2.0.3/2.0.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/jsonata/2.0.3/2.0.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>jsonata-js/jsonata (jsonata)</summary>

###
[`v2.0.4`](https://togithub.com/jsonata-js/jsonata/blob/HEAD/CHANGELOG.md#204-Maintenance-Release)

[Compare
Source](https://togithub.com/jsonata-js/jsonata/compare/v2.0.3...v2.0.4)

- Prevent writing to the object prototype or constructor (PR
[https://github.com/jsonata-js/jsonata/pull/676](https://togithub.com/jsonata-js/jsonata/pull/676))
- Add upper/lower presentation format for am/pm in fromMillis (PR
[https://github.com/jsonata-js/jsonata/pull/644](https://togithub.com/jsonata-js/jsonata/pull/644))
-   Various documentation additions and corrections

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 5am every weekday" in timezone
Europe/Berlin, Automerge - "after 10pm every weekday,before 5am every
weekday" in timezone Europe/Berlin.

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Never, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/solid-design-system/solid).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMjAuMiIsInVwZGF0ZWRJblZlciI6IjM3LjIyMC4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

fromMillis not handling PN picture string
2 participants