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

getZoneName calls correct builtin function, adds links to kernel code #24

Merged
merged 2 commits into from
Nov 23, 2023

Conversation

andrewMacmurray
Copy link
Owner

@andrewMacmurray andrewMacmurray commented Nov 22, 2023

Closes #23

Thanks to @lue-bird for spotting this.

This PR

  • Makes Time.getZoneName call the correct builtin function and adds some integration smoke tests
  • adds some links to the relevant / equivalent Elm Kernel code.

@andrewMacmurray andrewMacmurray merged commit 516ab3a into main Nov 23, 2023
4 checks passed
@andrewMacmurray andrewMacmurray deleted the fix-get-zone-name branch November 23, 2023 08:31
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.

ConcurrentTask.Time.getZoneName references wrong js function
1 participant