telegramdesktop / tdesktop

Telegram Desktop messaging app
https://desktop.telegram.org/
Other
25.88k stars 5.13k forks source link

[Feature request] Checking for updates #4643

Open eirnym opened 6 years ago

eirnym commented 6 years ago

Please, add checking for updates and updating. The popular method is to use Sparkle.framework to achieve this goal.

Expected behaviour

Tell us what should happen A menu item "Check for updates" in OS X menu panel and an option in preferences to enable or disable them.

Actual behaviour

Tell us what happens instead I have to check them manually all the time.

Configuration

Operating system: Mac OS X Version of Telegram Desktop: 1.2.17 Used theme: any

Logs: Insert logs here (if necessary)
john-preston commented 6 years ago

@eirnym There is autoupdate already, you can press Check for updates in Settings.

It is available only in the version from https://desktop.telegram.org, not from App Store.

stek29 commented 6 years ago

Version of Telegram Desktop: 2.1.17

Where did you even get this? It's not TDesktop, and it's not new Swift native macOS app -- it's a really really old app which is no longer maintained.

You can get tdesktop at https://desktop.telegram.org, and you can get new macOS native app at https://macos.telegram.org.

john-preston commented 6 years ago

@stek29 I thought he meant 1.2.17, but just misspelled 🤔

stek29 commented 6 years ago

@john-preston seems so indeed...

eirnym commented 6 years ago

I want to have a button to check them manually, without turning on autoupdates

stek29 commented 6 years ago

@eirnym open settings and click on check updates

eirnym commented 6 years ago

@stek29 there's no such button. if you don't believe me, I can to make a full screenshot of my settings

zrzut ekranu 2018-05-06 o 18 33 18

eirnym commented 6 years ago

@stek29 Please, send screenshot of your settings with this button

eirnym commented 6 years ago

@john-preston Is there possible to check updates manually on macOS?

john-preston commented 6 years ago

@eirnym What do you mean by "on macOS"? If the version from AppStore, then no, only through the AppStore app. If the version from https://desktop.telegram.org — go to Settings and press "Check for updates".

stek29 commented 6 years ago

@eirnym Check update automatically, click on check for updates, uncheck Update automatically if you want to.

eirnym commented 6 years ago

@stek29 Just after I enable autoupdate option it runs a procedure to check the update and update it right away despite if user wants it or not.

I want check an update and optionally download and install the update.

stek29 commented 6 years ago

@eirnym so you want "check for updates" to be available when autoupdate is disabled?

eirnym commented 6 years ago

Yes, I want just this button or a menu item (as the most common place for it)

eirnym commented 5 years ago

Is it too hard to implement this?

john-preston commented 5 years ago

@eirnym I don't see any reason to check for updates without updating the app. It has no information about the update like changelog or anything, just that the new version is available. And you have no information to make a decision about you wanting some particular new version or not. So I think that current way is enough. If you want a new version - you enable auto update, it downloads a new version and installs it. If you don't want to update right now - just disable autoupdate.

eirnym commented 5 years ago

I have control when I want to download a new version. Not every connection and time is good for autoupdate

eirnym commented 5 years ago

talking about release notes, you have them on Github, they could be the same for updates

anohren commented 4 years ago

I had a hard time figuring out whether this issue was 1. a feature request for an already existing feature, 2. a bug report for erroneously hiding the option to manually check for updates in the very situation where it becomes the most relevant, or 3. a feature request for having that button actually do what it says, i.e. "Check" for updates instead of checking and updating.

I'm going to assume it's the 2nd, since someone decided to close my report of that bug without explanation with a link to here.

@eirnym so you want "check for updates" to be available when autoupdate is disabled?

I know I do. That's the very situaion in which the option to manually checking for updates becomes relevant, which means hiding it would be a bug.

I don't see any reason to check for updates without updating the app. It has no information about the update like changelog or anything, just that the new version is available. And you have no information to make a decision about you wanting some particular new version or not

This is a fair point, which would support the following ideal behavior in the absence of a changelog when autoupdate is disabled:

  1. Literally checking for updates is done automatically.
  2. Choosing to download and install the update is then done through the click of a button.

So I think that current way is enough. If you want a new version - you enable auto update, it downloads a new version and installs it. If you don't want to update right now - just disable autoupdate.

The current way is defective. It is apparent that people won't find the option to manually update when autoupdate is disabled. It has been demonstrated by the existence of this issue, and with a screenshot.

That the task is still techincally possible to perform doesn't change the fact that the current design forces people to do something they don't want to do -- in this case enabling autoupdates -- in order to complete it. It also forces them to figure out that two UI elements that should reasonably be orthogonal to each other are in fact dependent on each other.

stale[bot] commented 3 years ago

Hey there!

This issue will be automatically closed in 7 days if there would be no activity. We therefore assume that the user has lost interest or resolved the problem on their own.

Don't worry though; if this is an error, let us know with a comment and we'll be happy to reopen the issue.

Thanks!

anohren commented 3 years ago

We therefore assume that the user has lost interest or resolved the problem on their own.

Since this is a bug report there is no problem that can be resolved by end users. It would therefore be much more reasonable to conclude that developers have lost interest, if anyone — not the users. There is no reason to close.

eirnym commented 3 years ago

I didn't lost an interest, update scheme I and @anohren mentioned is implemented a long list of open source software I know.

Now I'll discuss interaction with a user more than checking and update mechanism itself.

Option 1: Pop-up window as it's implemented in Sparkle.Framework for macOS applications. Little bit noisy for a user, but works well.

Option 2: Firefox if autoupdate is disabled, checks and possibly installs a new update in "about" dialog

Option 3: Google Chrome shows an icon on the top right corner if autoupdate is disabled or impossible for some reason.

Current behaviour of Telegram.app is to continue be silent even if I left autoupdate on but I revoke write permissions from a app folder.

My vote is for an option 3, but don't be silent and give a user a manual update option

stale[bot] commented 3 years ago

Hey there!

This issue was inactive for a long time and will be automatically closed in 30 days if there isn't any further activity. We therefore assume that the user has lost interest or resolved the problem on their own.

Don't worry though; if this is an error, let us know with a comment and we'll be happy to reopen the issue.

Thanks!

anohren commented 3 years ago

We therefore assume that the user has lost interest

No point mentioning it again; yes, let's say users have lost interest. At least one just did. Unsubscribing.

eirnym commented 3 years ago

Should I ping this ticket every 2 mounts with worthless comments?

Aokromes commented 3 years ago

Should I ping this ticket every 2 mounts with worthless comments?

well, it was 6 months, also, slate bot helps to find still opened tickets already fixed.

eirnym commented 3 years ago

Ok, I don't see it implemented in current version, so how it was fixed?

Zrzut ekranu 2021-04-23 o 22 07 00 Zrzut ekranu 2021-04-23 o 22 06 54
Aokromes commented 3 years ago

Ok, I don't see it implemented in current version, so how it was fixed?

Zrzut ekranu 2021-04-23 o 22 07 00 Zrzut ekranu 2021-04-23 o 22 06 54

"slate bot helps to find still opened tickets already fixed." i don't means this issue is fixed.

eirnym commented 3 years ago

I see. I think it shows "no new comments" after some time, not what is "fixed".

eirnym commented 3 years ago

Even I'm able to make this fixed, there's a lot of reports, that it's hard to build on Linux or macOS because I need to have very specific set of libraries and very specific versions of them in very specific places. Because of this Telegram packages were removed from linux repos.

Is there any guidance on that matter?

Aokromes commented 3 years ago

I see. I think it shows "no new comments" after some time, not what is "fixed".

people comments if it's unfixed, or if it's fixed.

ilya-fedin commented 3 years ago

Because of this Telegram packages were removed from linux repos.

AFAIK it's still in all repos it was previously in

eirnym commented 3 years ago

this "stale" bot wrote why it marks issue as "stale". I don't know why you think it's the same as "fixed".

If you want to implement it, go ahead! I'll be happy with his feature request implemented.

ilya-fedin commented 3 years ago

I don't know why you think it's the same as "fixed".

I don't know why you think we think it's the same as "fixed"

ilya-fedin commented 3 years ago

If you want to implement it, go ahead!

Not really. The fact feature requests exists doesn't mean it will be ever implemented.

Aokromes commented 3 years ago

this "stale" bot wrote why it marks issue as "stale". I don't know why you think it's the same as "fixed".

If you want to implement it, go ahead! I'll be happy with his feature request implemented.

... slate: waiting for answer from people on issue to know if it's already fixed or not. we don't think slate means it's fixed. with the help of slate bot we found some issues already fixed and closed, others remains opened.

ilya-fedin commented 3 years ago

In case of bug reports, stalebot helps to find bugs that were disappeared on their own or fixed, but issues were forgotten In case of feature requests, stalebot helps to find feature requests no one really wants anymore or that were implemented, but issues were forgotten

tooomm commented 3 years ago

In case of feature requests, stalebot helps to find feature requests no one really wants anymore

with the help of slate bot we found some issues already fixed and closed, others remains opened.

And many got also closed unfixed... because people are annoyed by the bot and the constant need to spam issues with senseless comments and useless notifications. There are issues with more bot and bump comments than actual content. 🙄

Don't confuse that with people not demanding certain feature anymore, please.

ilya-fedin commented 3 years ago

Well if people don't answer then it means they don't really want a fix xD If they want a soon fix they can contribute it themselves, otherwise they should wait and pong when the bot pings. ¯_(ツ)_/¯

ilya-fedin commented 3 years ago

Without the bot there will again be >1k issues with a lot of duplicates and it would be really hard to find something valuable for anyone who wants to find something, again.

eirnym commented 3 years ago

There's a problem with notifications. Labels don't generate them. So "pong back" doesn't work even if I'd be subscribed to or be a member of the whole project