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

feat: Outlook Calendar Integration #27922

Merged
merged 93 commits into from
Jul 4, 2023
Merged

Conversation

dougfabris
Copy link
Member

@dougfabris dougfabris commented Feb 1, 2023

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

VC-29

@pierre-lehnen-rc pierre-lehnen-rc force-pushed the new/outlook-calendar-integration branch from d22832b to d62dec3 Compare March 6, 2023 18:29
@dougfabris dougfabris changed the title [NEW] Outlook Calendar Integration feat: Outlook Calendar Integration Mar 14, 2023
@codecov
Copy link

codecov bot commented Mar 21, 2023

Codecov Report

Merging #27922 (9638575) into develop (e05e581) will decrease coverage by 0.06%.
The diff coverage is 40.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #27922      +/-   ##
===========================================
- Coverage    46.65%   46.60%   -0.06%     
===========================================
  Files          693      693              
  Lines        12934    12946      +12     
  Branches      2241     2242       +1     
===========================================
- Hits          6035     6034       -1     
- Misses        6574     6587      +13     
  Partials       325      325              
Flag Coverage Δ
e2e 46.57% <40.00%> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@dougfabris dougfabris force-pushed the new/outlook-calendar-integration branch from f289e31 to acc9bd5 Compare April 3, 2023 23:46
ggazzo
ggazzo previously approved these changes Jun 30, 2023
@scuciatto scuciatto removed the request for review from a team June 30, 2023 11:22
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Jun 30, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Jun 30, 2023
KevLehman
KevLehman previously approved these changes Jun 30, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Jul 3, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Jul 3, 2023
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: ready to merge PR tested and approved waiting for merge labels Jul 4, 2023
@ggazzo ggazzo merged commit c10137e into develop Jul 4, 2023
@ggazzo ggazzo deleted the new/outlook-calendar-integration branch July 4, 2023 14:40
@way2song
Copy link

way2song commented Aug 3, 2023

Hello team! In what release is it planned to implement this?

@way2song
Copy link

way2song commented Aug 3, 2023

image

There is no such thing in 6.3.0

@dougfabris
Copy link
Member Author

dougfabris commented Aug 8, 2023

Hi @way2song

It's going to be available in 6.3.0 as an enterprise feature!


Questions? Help needed? Feature Requests?

  • Join our Open Server in the #support channel and feel free to raise a question
  • Join our Community Forum and search/create a post there

@way2song
Copy link

way2song commented Aug 9, 2023

Hi @way2song

It's going to be available in 6.3.0 as an enterprise feature!

Questions? Help needed? Feature Requests?

  • Join our Open Server in the #support channel and feel free to raise a question
  • Join our Community Forum and search/create a post there

Hi good.
This is in 6.3.0, but it doesn't work correctly. In the electron version, I had to replace the 'X-Auth-Token': token,
To a token that receives globally
'X-Auth-Token': userAPIToken,
It works, but does not update meeting information. There are new errors.
In fact, there is no description of how it should work. Nowhere is it indicated that after authorization, authorization data is stored in the electron.
It is also not described that in the URL field you need to specify only the server address, and not the full path to ews
In any case, I had to look at the code.
The calendar is displayed only for group chats, and you can not call it for personal correspondence.
You cannot make an appointment or invite a user.
A lot needs to be improved.

@dougfabris
Copy link
Member Author

Hi @way2song

Thanks for the feedback, I will report it to the product team!


Questions? Help needed? Feature Requests?

  • Join our Open Server in the #support channel and feel free to raise a question
  • Join our Community Forum and search/create a post there

@way2song
Copy link

Hi @way2song

Thanks for the feedback, I will report it to the product team!


Questions? Help needed? Feature Requests?

  • Join our Open Server in the #support channel and feel free to raise a question

  • Join our Community Forum and search/create a post there

Hello, any news?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
squad: video conferencing stat: ready to merge PR tested and approved waiting for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants