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.

Sign up for GitHub

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

Jump to bottom

Do Not Merge – Only here to trigger CI on https://github.com/wordpress-mobile/WordPress-Android/pull/15570 #15615

Merged
merged 35 commits into from Nov 24, 2021
Merged

Do Not Merge – Only here to trigger CI on https://github.com/wordpress-mobile/WordPress-Android/pull/15570 #15615

merged 35 commits into from Nov 24, 2021

Conversation

antonis
Copy link
Member

@antonis antonis commented Nov 24, 2021

Do Not Merge – Only here to trigger CI on #15570

Ovi Trif added 30 commits November 13, 2021 14:48
Add new option to published post to copy the url to clipboard
24417d1
Update unit tests for post actions
323fdbd
Add gridicon-link and use it for the copy url post button
d0c7753
Call invoke on triggerPostListAction for copy url
470a605
Extract clipboard manager as context extension
30117ab
Log clipboard manager exception post exception
fa4c053
Remove unused imports in PostListAction
471f7f0
Use post id as clipData label since it's only use is for developers
889d7f8 
- https://stackoverflow.com/questions/33207809/what-exactly-is-label-parameter-in-clipdata-in-android
Show toast notification when the post link is copied
079352f
Refactor: Use addIf for BUTTON_COPY_URL to fix detekt err
d48dd95
Update detekt baseline
a128bfa
Update release notes
f556b13
Update release notes with additional newline
c6f497f
Update release notes with correct link to pr
4d83750
Merge branch 'develop' into issue/13292-copy-published-post-url
d54969b
Fix typo in statusesDelimiter
f548f16
Add proper annotation for one test in PostListItemUiStateHelperTest
4298845
Extract function to add delete or trash action button
8fac44d
Extract function to add view or preview button
6e3550e
Use conditional instead of addIf extension to add copyUrl button
6551903
Fix redundant qualifier name errors in PostListItemUiStateHelper
48b7432
Add missing post buttons test for case when stats are not supported
0f2ad9e
Use optional call to set primary clip and throw NPE otherwise
60c19e5
Explain why we handle generic exceptions in copy-to-clipboard code
7398ab3
Merge branch 'develop' into issue/13292-copy-published-post-url
67dd49d
Move release note update to newest milestone
e62023d
Add reference to chromium pr to catch for generic clipboard exceptions
6ce9767
Merge branch 'develop' into issue/13292-copy-published-post-url
f2d5238
Merge branch 'develop' into issue/13292-copy-published-post-url
ab2641d
Show copy link action for posts of any status except trashed
a1bfa31
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Nov 24, 2021 •
edited

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

@peril-wordpress-mobile
Copy link

You can test the changes on this Pull Request by downloading the APKs:

@antonis antonis merged commit bf3b212 into develop Nov 24, 2021
@antonis antonis deleted the issue/13292-copy-published-post-url branch November 24, 2021 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@antonis