controller: getRead + all_items is now unread type #1380
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Sean Molenaar <[email protected]> Signed-off-by: Benjamin Brahmer <[email protected]>
SMillerDev
approved these changes
May 27, 2021
Codecov Report
@@ Coverage Diff @@
## master #1380 +/- ##
============================================
+ Coverage 90.41% 90.43% +0.02%
- Complexity 761 764 +3
============================================
Files 66 66
Lines 2806 2803 -3
============================================
- Hits 2537 2535 -2
+ Misses 269 268 -1
Continue to review full report at Codecov.
|
Grotax
added a commit
that referenced
this pull request
Jun 1, 2021
Changed - Allow installation on Nextcloud v22 - Remove deprecated API endpoints and occ comand (#935) - /api/v1-2/user - /api/v1-2/user/avatar - ./occ news:updater:all-feeds Fixed - allow calling `/items?getRead=false` without a feed/folder (#1380 #1356) - newestId does not return newest ID but last updated (#1339) Signed-off-by: Benjamin Brahmer <[email protected]>
Merged
Grotax
added a commit
that referenced
this pull request
Jun 2, 2021
Changed - Allow installation on Nextcloud v22 - Remove deprecated API endpoints and occ comand (#935) - /api/v1-2/user - /api/v1-2/user/avatar - ./occ news:updater:all-feeds Fixed - allow calling `/items?getRead=false` without a feed/folder (#1380 #1356) - newestId does not return newest ID but last updated (#1339) Signed-off-by: Benjamin Brahmer <[email protected]>
Grotax
added a commit
that referenced
this pull request
Jul 1, 2021
There are no additional changes compared to the latest beta. Changed - News now requires a 64bit OS - v2 API implementation (folder part) - Implemented sharing news items between nextcloud users (#1191) - Updated the news items table in DB to include sharer data (#1191) - Added route for sharing news items (#1191) - Added share data in news items serialization (#1191) - Added tests for the news items share feature (#1191) - Added sharing articles with nextcloud users (#1217) - Added sharing articles on social media (Facebook, Twitter) or mail (#1217) - Allow installation on Nextcloud v22 - Remove deprecated API endpoints and occ command (#935) - /api/v1-2/user - /api/v1-2/user/avatar - ./occ news:updater:all-feeds - added feed search (#1402) Fixed - allow calling `/items?getRead=false` without a feed/folder (#1380 #1356) - newestId does not return newest ID but last updated (#1339) - removed reference for deleted repair-steps (#1399) - Fix NotNullConstraintViolation when sharing news items with users (#1406) Signed-off-by: Benjamin Brahmer <[email protected]>
Merged
Grotax
added a commit
that referenced
this pull request
Jul 4, 2021
There are no additional changes compared to the latest beta. Changed - News now requires a 64bit OS - v2 API implementation (folder part) - Implemented sharing news items between nextcloud users (#1191) - Updated the news items table in DB to include sharer data (#1191) - Added route for sharing news items (#1191) - Added share data in news items serialization (#1191) - Added tests for the news items share feature (#1191) - Added sharing articles with nextcloud users (#1217) - Added sharing articles on social media (Facebook, Twitter) or mail (#1217) - Allow installation on Nextcloud v22 - Remove deprecated API endpoints and occ command (#935) - /api/v1-2/user - /api/v1-2/user/avatar - ./occ news:updater:all-feeds - added feed search (#1402) Fixed - allow calling `/items?getRead=false` without a feed/folder (#1380 #1356) - newestId does not return newest ID but last updated (#1339) - removed reference for deleted repair-steps (#1399) - Fix NotNullConstraintViolation when sharing news items with users (#1406) Signed-off-by: Benjamin Brahmer <[email protected]>
Neo11
pushed a commit
to Neo11/news
that referenced
this pull request
May 28, 2022
Changed - Allow installation on Nextcloud v22 - Remove deprecated API endpoints and occ comand (nextcloud#935) - /api/v1-2/user - /api/v1-2/user/avatar - ./occ news:updater:all-feeds Fixed - allow calling `/items?getRead=false` without a feed/folder (nextcloud#1380 nextcloud#1356) - newestId does not return newest ID but last updated (nextcloud#1339) Signed-off-by: Benjamin Brahmer <[email protected]>
Neo11
pushed a commit
to Neo11/news
that referenced
this pull request
May 28, 2022
There are no additional changes compared to the latest beta. Changed - News now requires a 64bit OS - v2 API implementation (folder part) - Implemented sharing news items between nextcloud users (nextcloud#1191) - Updated the news items table in DB to include sharer data (nextcloud#1191) - Added route for sharing news items (nextcloud#1191) - Added share data in news items serialization (nextcloud#1191) - Added tests for the news items share feature (nextcloud#1191) - Added sharing articles with nextcloud users (nextcloud#1217) - Added sharing articles on social media (Facebook, Twitter) or mail (nextcloud#1217) - Allow installation on Nextcloud v22 - Remove deprecated API endpoints and occ command (nextcloud#935) - /api/v1-2/user - /api/v1-2/user/avatar - ./occ news:updater:all-feeds - added feed search (nextcloud#1402) Fixed - allow calling `/items?getRead=false` without a feed/folder (nextcloud#1380 nextcloud#1356) - newestId does not return newest ID but last updated (nextcloud#1339) - removed reference for deleted repair-steps (nextcloud#1399) - Fix NotNullConstraintViolation when sharing news items with users (nextcloud#1406) Signed-off-by: Benjamin Brahmer <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rebase of #1356 merged into stable15
Signed-off-by: Sean Molenaar [email protected]
Signed-off-by: Benjamin Brahmer [email protected]