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

BPA Rule Updates #247

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

BPA Rule Updates #247

wants to merge 8 commits into from

Conversation

kerski
Copy link
Contributor

@kerski kerski commented Feb 2, 2024

I noticed the INTEGER_FORMATTING rule was falsely identifying Date-based DAX measures against this rule.

Also description for Provide format string for "Date" and "Month" column rules had the same description. I fixed the description to reference the correct name.

kerski and others added 4 commits February 1, 2024 21:33
Update to description of "Provide format string for Date columns
Handle empty files.  For example, an empty DAX Query View file.
@kerski
Copy link
Contributor Author

kerski commented May 1, 2024

I noticed an error on Import-FabricItem and Import-FabricItems with the following error: Exception calling "ToBase64String" with "1" argument(s): "Value cannot be null. (Parameter 'inArray')" This can occur if someone saves a pbip file with an empty DAX Query View file. I added a ternary operator to check fileContent is not null before trying to call the "ToBase64String" function.

@kerski kerski changed the title BPA Rule Updates BPA Rule Updates and FabricPS-PBIP.psm1 update to check for empty files. May 1, 2024
John Kerski added 2 commits May 1, 2024 12:35
Handle empty files.  For example, an empty DAX Query View file.
@kerski kerski changed the title BPA Rule Updates and FabricPS-PBIP.psm1 update to check for empty files. BPA Rule Updates May 1, 2024
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.

1 participant