Skip to content

Share reports via email or share sheet#6

Open
Workshed wants to merge 6 commits into
masterfrom
feature/otherSharing
Open

Share reports via email or share sheet#6
Workshed wants to merge 6 commits into
masterfrom
feature/otherSharing

Conversation

@Workshed

Copy link
Copy Markdown
Owner

Allow sharing via email or via anything contained in the share sheet. This could be AirDrop, Slack, iMessage, etc etc.

The report itself is a zip file containing the original screenshot, an annotated screenshot (if created) as well as any extra details provided.

Comment thread Spot/Spot.swift Outdated
}

static func reportEmailMessageBody() -> String {
// static func reportEmailMessageBody() -> String {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove commented code?

Comment thread Spot/Spot.swift Outdated
}
if let additionalInformation = Spot.delegate?.additionalEmailContent() {
bodyText += "Additional information: \(additionalInformation)"
reportInfo += "Additional information: \(additionalInformation)"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worth putting the \n on the end in readiness for more being added at any time?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants