Don't show boosts and likes for "all" and "local" in timelines #781

Merged
elegaanz merged 1 commits from dedup-articles into master 4 years ago
elegaanz commented 4 years ago (Migrated from github.com)

The fix was quite easy actually 😅 I think it is just that it wasn't clear enough for us that local/all shouldn't match boosts and likes, but only new posts.

Fixes #711

The fix was quite easy actually :sweat_smile: I think it is just that it wasn't clear enough for us that `local`/`all` shouldn't match boosts and likes, but only new posts. Fixes #711
igalic (Migrated from github.com) approved these changes 4 years ago
codecov[bot] commented 4 years ago (Migrated from github.com)

Codecov Report

Merging #781 into master will decrease coverage by 0.01%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #781      +/-   ##
==========================================
- Coverage   39.08%   39.07%   -0.02%     
==========================================
  Files          73       73              
  Lines        9736     9736              
  Branches     2227     2227              
==========================================
- Hits         3805     3804       -1     
+ Misses       4879     4878       -1     
- Partials     1052     1054       +2     
# [Codecov](https://codecov.io/gh/Plume-org/Plume/pull/781?src=pr&el=h1) Report > Merging [#781](https://codecov.io/gh/Plume-org/Plume/pull/781?src=pr&el=desc) into [master](https://codecov.io/gh/Plume-org/Plume/commit/ef70cb93e6d9457355bce4f6dae485c700bb07c6&el=desc) will **decrease** coverage by `0.01%`. > The diff coverage is `50.00%`. ```diff @@ Coverage Diff @@ ## master #781 +/- ## ========================================== - Coverage 39.08% 39.07% -0.02% ========================================== Files 73 73 Lines 9736 9736 Branches 2227 2227 ========================================== - Hits 3805 3804 -1 + Misses 4879 4878 -1 - Partials 1052 1054 +2 ```
Chostakovitch commented 4 years ago (Migrated from github.com)

Marvellous ! Thank you ! :D

Marvellous ! Thank you ! :D

Reviewers

The pull request has been merged as 297d9fcf40.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b dedup-articles master
git pull origin dedup-articles

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff dedup-articles
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#781
Loading…
There is no content yet.