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

MM-56819: Introduce --local mode in MMCTL to handle user preferences. #26972

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

Conversation

kaoski
Copy link
Contributor

@kaoski kaoski commented May 8, 2024

Summary

This commit introduces support for --local mode in MMCTL for managing user preferences. The following changes have been added:

  1. API endpoints for user preferences are registered with the local router.
  2. Test cases have been added to ensure functionality for both System Admins and --local mode.

Note: As there is no authentication/authorization required in --local mode, @mattermod suggested omitting session checks from the API handlers. But we are already bypassing session checks for local mode requests as detailed here. Hence, I have retained the existing session checks in place.

Ticket Link

Github Issue: #26864
Jira Ticket: https://mattermost.atlassian.net/browse/MM-56819

Release Note

Added --local mode support in MMCTL to handle user preferences.

@mm-cloud-bot
Copy link

@kaoski: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

I understand the commands that are listed here

@mattermost-build
Copy link
Contributor

Hello @kaoski,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@mattermost-build
Copy link
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants