-
Notifications
You must be signed in to change notification settings - Fork 4
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
IYY-266: Ensure Microsoft BI embeds work #810
Conversation
Created multidev environment pr-810 for yalesites-platform. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@codechefmarc This is awesome. I did have a question; is that Sign In address available to all to log in and see? I'm just thinking of users are going to test out the PowerBI with what's in the help modal, we might want the other there since they can see the output immediately without logging in?
Or maybe we can find a way to express both examples somehow? I think at the moment it only allows one in the embed patterns though doesn't it.
Passing it as it does great, but wanted to put that here; maybe we revert the help back to the old one so they don't have to worry about signing in to see it in action?
Yeah, I agree - can we get one of the new links that is publicly available? Perhaps someone with Power BI access can create one that is public and send that over and I'll update the code? And yeah, I tried to put two examples but it looked too gnarly. We can use the original example, for sure, but if they are moving to the new URL, I'd rather have the new one in there. |
Ah great idea. Wonder who owns that original? I bet they could make it. LOL @miketullo95 Do you know who might have made that embed? |
that is a great question. We can ask that in our internal meeting tomorrow to find out |
Hi team, Testing this out I realized an extra step we were missing in generating the code from PowerBI... The current instructions are:
Those instructions generate an embed code like:
Which requires Login to gain access. However, in Power BI, if you go to File menu, select Embed report > Publish to web (public) That will give you an embed code like our original, and which obviously doesn't require login (it's public):
So, essentially we've now created the ability to embed a report which requires you to be logged in, which could be useful. I think the main use case is for public accessible reports though, so we'll want to update those help instructions to add in that changed step. |
So I think if we could keep what we have, and adjust the help text:
And then include two examples, one for each. |
@duncancm9 (strange I still can't make you a reviewer?) |
🎉 This PR is included in version 1.12.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
IYY-266: Ensure Microsoft BI embeds work
Description of work
/view
and/reportEmbed
)Functional testing steps:
https://app.powerbi.com/view?r=eyJrIjoiYzQ1ODA0ZjEtZjc5YS00OTgyLWIzOTItNmJmNDY2YmRiODQ2IiwidCI6ImRkOGNiZWJiLTIxMzktNGRmOC1iNDExLTRlM2U4N2FiZWI1YyIsImMiOjF9&pageName=ReportSection2ac2649f17189885d376