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

Enhancement: Allow Outlook Cloud connector to specify client emails instead of defaulting to all users #2843

Open
ilyasabdellaoui opened this issue Sep 23, 2024 · 2 comments · May be fixed by #2844

Comments

@ilyasabdellaoui
Copy link

ilyasabdellaoui commented Sep 23, 2024

Problem Description

The current Outlook Cloud connector attempts to fetch all users by default. This can lead to performance issues and unnecessary load, especially in environments where only a few client emails are relevant for the integration.

Proposed Solution

Introduce a configuration option in the Outlook Cloud connector to specify client emails (as a comma-separated list) to fetch only the relevant user data. This change will enhance performance by reducing the number of users fetched and improve the connector's efficiency.

@ilyasabdellaoui ilyasabdellaoui added the enhancement New feature or request label Sep 23, 2024
@ilyasabdellaoui ilyasabdellaoui linked a pull request Sep 23, 2024 that will close this issue
9 tasks
@seanstory
Copy link
Member

@ilyasabdellaoui thanks for filing! This is a good feature request. CC @danajuratoni for prioritization - could be something we ask team:external to take a look at?

@ilyasabdellaoui
Copy link
Author

just a quick note that I’ve opened a PR #2844 that addresses this feature request.

@ilyasabdellaoui ilyasabdellaoui changed the title Enhancement: Allow Outlook Cloud connector to specify client emails instead of defaulting to top 999 users Enhancement: Allow Outlook Cloud connector to specify client emails instead of defaulting to all users Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants