Skip to content

Conversation

tormodfj
Copy link
Contributor

Details of Changes

Validation
How did you test these changes?

  • Built and ran the app
  • Tested the changes for accessibility

Screenshots (optional)
Jottacloud

@tormodfj
Copy link
Contributor Author

Apologies for circumventing the process of creating and discussing this as an issue, before submitting a PR. The required code was so trivial that basically no time was wasted, should you decide to reject this PR.

Copy link
Member

@gave92 gave92 left a comment

Choose a reason for hiding this comment

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

Never heard of jottacloud xD but this PR looks good, thanks!

@gave92 gave92 added the ready to merge Pull requests that are approved and ready to merge label Jan 17, 2022
@yaira2 yaira2 merged commit 7451058 into files-community:main Jan 17, 2022
@yaira2
Copy link
Member

yaira2 commented Jan 17, 2022

@tormodfj can you add your agreement to #7625?

@tormodfj tormodfj deleted the jottacloud-detector branch January 17, 2022 14:38
@tormodfj
Copy link
Contributor Author

@tormodfj can you add your agreement to #7625?

Already done 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Pull requests that are approved and ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants