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

OWA: Teams Meetings #455

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

NeilRashbrook
Copy link
Collaborator

I see you restored isOnline!

Creating an event in the correct calendar is still awkward.

  • Open Calendar view
  • Click +
  • Select the correct calendar from the picker
  • Click X
  • Click +
  • Now create the event.

Note that if you don't add a description to the event before turning it into a Teams meeting, you won't see the Teams meeting description, because it's in text, not HTML.

@NeilRashbrook NeilRashbrook self-assigned this Feb 24, 2025
@benbucksch
Copy link
Collaborator

benbucksch commented Feb 25, 2025

I see you restored isOnline!

You're welcome.

Click X Click +

That means the account picker doesn't work in this dialog as it should. Feel free to file a bug report with reproduction in these cases, as new issue.

I'll take a look at it.

if you don't add a description to the event before turning it into a Teams meeting, you won't see the Teams meeting description, because it's in text, not HTML.

That means that the Event.descriptionText/HTML doesn't work as it should. They should automatically sychronize. (Unless the server messes us up, but even then, we still need to deal with it.)

Can you please file a bug about it, with reproduction steps, what exactly happebs etc?

Copy link
Collaborator

@benbucksch benbucksch left a comment

Choose a reason for hiding this comment

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

Can you please make sure that whenever you are reading from the network, you always use the most specific sanitize.*() function available? E.g. sanitize.url() instead of sanitize.string() etc.

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.

2 participants