Skip to content

issue-563 Attaching screenshot causes error#602

Merged
tomholub merged 2 commits intomasterfrom
bugfix/issue-563-attaching-screenshot-causes-error
Oct 4, 2021
Merged

issue-563 Attaching screenshot causes error#602
tomholub merged 2 commits intomasterfrom
bugfix/issue-563-attaching-screenshot-causes-error

Conversation

@ekievsky
Copy link
Contributor

@ekievsky ekievsky commented Oct 4, 2021

This PR allows to select video, removes permissions to photos, fixed limited access issue

close #563
close #547
close #564


Tests :

  • Does not need tests (refactor only, docs or internal changes)

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@ekievsky ekievsky changed the title Fixed bug with limited access; Allowed to send video om attachments; issue-563 Attaching screenshot causes error Oct 4, 2021
Copy link
Collaborator

@tomholub tomholub left a comment

Choose a reason for hiding this comment

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

Looks good

@tomholub tomholub enabled auto-merge (squash) October 4, 2021 21:17
@tomholub tomholub merged commit 3c777f5 into master Oct 4, 2021
@tomholub tomholub deleted the bugfix/issue-563-attaching-screenshot-causes-error branch October 4, 2021 21:27
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.

cannot attach a video attaching a screenshot (instead of a photo) causes error can't select second photo attachment

2 participants