mattermost/mattermost-plugin-mscalendar

Handle `EmptySuggestionReason` field value in `FindMeetingTimes()` method

Open

#27 opened on Jan 29, 2020

View on GitHubContributor guide
 (0 comments) (0 reactions) (0 assignees)Go (35 forks)auto 404
Difficulty/1:EasyHacktoberfestHelp WantedTech/GoType/EnhancementUp For Grabs

Repository metrics

Stars
 (22 stars)
PR merge metrics
 (PR metrics pending)

Description

Summary

The FindMeetingTimes() method in server/remote/msgraph/find_meeting_times.go needs to handle the case where meetingsOut.EmptySuggestionReason field is a non-nil value and report the error.

Also, an error case where this error occurs on Microsoft Calendar's side needs to be discovered for testing.

Ticket Link

https://mattermost.atlassian.net/browse/MM-21348

Contributor guide

Research direction
Inspect the FindMeetingTimes() method in server/remote/msgraph/find meeting times.go to handle EmptySuggestionReason field. Look for existing tests or create a test scenario where Microsoft Calendar returns a non nil EmptySuggestionReason.
Tech stack
go
Domain
backend
Issue type
Bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Go
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75