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

[MI-2705]:Added summary for status sync job #4

Merged
merged 6 commits into from
Feb 6, 2023
Merged

Conversation

Kshitij-Katiyar
Copy link
Collaborator

@Kshitij-Katiyar Kshitij-Katiyar added the server This is server related label Feb 2, 2023
server/jobs/status_sync_job.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Outdated Show resolved Hide resolved
server/mscalendar/availability.go Show resolved Hide resolved
@Kshitij-Katiyar Kshitij-Katiyar merged commit f7efeba into MM-244 Feb 6, 2023
@manojmalik20 manojmalik20 deleted the MI-2705 branch February 6, 2023 15:10
Kshitij-Katiyar added a commit that referenced this pull request Apr 12, 2023
…ppens for a single user" by added additional logging (mattermost#245)

* [MI-2664]:Fixed issue mattermost#244 on ms calender to add additional logging (#2)

* [MI-2664]:Fixed issue mattermost#244 on ms calender to add additional logging

* [MI-2664]:Added comment for skipping user

* [MI-2664]:Fixed unit test cases

* [MI-2664]:Fix lint errors

* [MI-2664]:Added conditional logging

* [MI-2664]:Added constants for limit

* [MI-2664]:Fixed review comments

* [MI-2664]:Fixed review comments

* [MI-2687]:Fixed review comments given by mattermost team on issue#244 of mscalender plugin (#3)

* [MI-2687]:Fixed review comments given by mattermost team on issue mattermost#244 of mscalender plugin

* [MI-2687]:Fixed unit test cases

* [MI-2687]:Fixed review fixes

* [MI-2705]:Added summary for status sync job (#4)

* [MI-2705]:Added summary for status sync job

* [MI-2705]:Fixed review comments

* Fixed lint errors (mattermost#243)

* [MI-2705]:Fixed review comments

* [MI-2705]:Fixed review comments

---------

Co-authored-by: Manoj Malik <manoj@brightscout.com>

* [MM-244]:Fixed lint errors

* [MM-244]:Added a check for summary to be nill

* [MM-244]:Added addtional check for user error

* [MM-244]:Fixed lint errors

* [MM-244]:Fixed review comments

---------

Co-authored-by: Manoj Malik <manoj@brightscout.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
server This is server related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants