chore: remove refresh option #2350
Conversation
rebelchris
requested review from
omBratteng,
ilasw,
sshanzel,
nensidosari and
AmarTrebinjac
| """ | ||
| Force refresh the feed | ||
| """ | ||
| refresh: Boolean = false | ||
|
|
There was a problem hiding this comment.
@rebelchris just to double check that frontend does not send this in any way 🙏 (if you did not already)
There was a problem hiding this comment.
I'm only removing it in the next app release here:
dailydotdev/apps#3687
There was a problem hiding this comment.
And extension, does it request it?
There was a problem hiding this comment.
Yeah the feeds do request it, so we can wait for adoption first to be safe I guess :)
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
|
@rebelchris time to merge this? 🙈 |
|
@omBratteng Think adoption is still low, considering also we have all the firefox users who can't update 😢 |
|
:lolsob: |
|
Maybe now? 👀 |
…move-refresh # Conflicts: # src/schema/feeds.ts
|
@capJavert The biggest issue is still the Firefox extension that never got updated. |
|
@rebelchris no, let's leave it, its not a drastic thing. |
We actually no longer use this.