Skip to content

Function to return the current local time for the requested US time zone using daylight saving rules.#40

Open
pawarbi wants to merge 4 commits into
daxlib:mainfrom
pawarbi:main
Open

Function to return the current local time for the requested US time zone using daylight saving rules.#40
pawarbi wants to merge 4 commits into
daxlib:mainfrom
pawarbi:main

Conversation

@pawarbi

@pawarbi pawarbi commented Sep 24, 2025

Copy link
Copy Markdown

Many times you need to show the local refresh times on the report. This DAX UDF will help return US timzezone when the model is refreshed.

Example usage

latest_us_pacifictime = GetLocalTimel( "uspacific" )

Comment thread packages/g/getlocaltime/0.1.0/icon.svg
@@ -0,0 +1,12 @@
{
"$schema": "https://raw.githubusercontent.com/sql-bi/daxlib/refs/heads/main/schemas/manifest/1.0.0/manifest.1.0.0.schema.json",
"id": "GetLocalTime",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you please rename the package using a hierarchical nomenclature? For example, something like PawarBI.TimeZone. For more details, see point 3 (Hierarchical Nomenclature) in our naming conventions: https://docs.daxlib.org/contribute/naming-conventions. Thanks!

Suggested change
"id": "GetLocalTime",
"id": "PawarBI.TimeZone",

@albertospelta albertospelta added the pr: author feedback required Pull Request is waiting for the author to respond or apply requested changes before merge. label Sep 24, 2025

@pawarbi pawarbi left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

  • replaced icon with png
  • updated namspace with GetTime.Local

@albertospelta

Copy link
Copy Markdown
Member

Thank you for updating the icon to PNG! However, it looks like the package name is still GetLocalTime.
Could you please update it as discussed in this comment? Thanks

@albertospelta albertospelta added the invalid This doesn't seem right label Oct 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

invalid This doesn't seem right pr: author feedback required Pull Request is waiting for the author to respond or apply requested changes before merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants