UncleGoogle / galaxy-integration-humblebundle

Humble Bundle integration for GOG Galaxy 2.0
GNU General Public License v3.0
184 stars 19 forks source link

Humble Choice games listed as all the others. #147

Closed nosyn00b closed 3 years ago

nosyn00b commented 3 years ago

I'd like to have an option to have all Humble Games that are part of an Humbel Choice listed with all the others under Humble-Bundle" list, as they were simply owned games. Redeemd games should not be listed (currently also redeemed games are listed in subscriptions, I think by design)

UncleGoogle commented 3 years ago

Hello. So you claim that deselection of "show reveled keys" option does not work for your choice games? Does it overlap with https://github.com/UncleGoogle/galaxy-integration-humblebundle/issues/123 ?

Please help me understand it deeply. There are 3 states in case of Choice: 1 game possible to be chosen 2 game chosen but key not revealed 3 game chosen and key revealed

IIRC currently 2 and 3 are shown in owned_games and you can change the behavior by manipulating "show keys" option:

nosyn00b commented 3 years ago

Thanks UG for the clarification about the three possible states and showing only 2+3 games on "owned games". Simply Billiant.

I wrongly supposed that all games part of a subscription was shown in subscriptions even in 2 & 3 state! I will check better the behavoiur for case 3 when subscriptions get back to work for me (currently for some reason no subscription games apart the trove are shown and also no option is shown in the settings ). Will give a peek forking....

In any case my request was to have an option to have the games with state 1 in the list of the owned games, so that they can be searchable and displayed with all the others owned games, perhapse with a dedicated "subscription" tag.

UncleGoogle commented 3 years ago

ok, I see. Perhaps games with state 1 ARE searchable in the current implementation

image

(surpassingly even as "Owned games" despite they are returned as subscription game and NOT as owned game)

But is guess they are not dispalying for you becasue of #146

nosyn00b commented 3 years ago

I dicovered that you can create an "All games" collection, selecting both filters owned and subscriptions image so you can search all your games in a single list as I wanted (no total games counter but it does not matter)!

PS: it seams that games in status 3 are still listed in subscription. I currently do not have games in status 2 so I cannot try.

UncleGoogle commented 3 years ago

Oh nice worth added to FAQ in readme. If you could make PR would be awesome.

Subscription interface returns all: 1,2,3 + extraxes OR only 2,3 +extraxes if you have no choices left in current month.

Subscription games may not be 1:1 with owned choices returned by humble api. The latter may contain more products/keys, the former is just like scrapping the choce month.

nosyn00b commented 3 years ago

Ok, I'll PR with a new FAQ chapter in readme asap.

Thanks!

PS: I confirm that in my case deselection of "show reveled keys" does not work for choice, I always see status 3 games in the list in choice.

UncleGoogle commented 3 years ago

PS: I confirm that in my case deselection of "show reveled keys" does not work for choice, I always see status 3 games in the list in choice.

yeah, it does not work Choice games shown in Subscription tab. It need design rethinking, as this option is misleading.

nosyn00b commented 3 years ago

Hi.

I updated the readme with a FAQ section (see PR), but i saw you're using gyazo as image source, an the gyazo client gives me virus warning (most probably is a false positive).

Anyway I think using an internal resource reference to the repository content is the most clean way to refer images in the readme files, even if has the drawback of increasing your repository size (Have a look at my Fanatical repository to see what i mean)

I temporarly used the images on this issue as image source. For example the following one.

image

Let me know what you think on this (having mixed image reference type is not very elegant).

UncleGoogle commented 3 years ago

totally agree, spawned issue to not forget

UncleGoogle commented 3 years ago

Ok, so in the future it would be nice to have additional filter in humble setting window that allows to:

So either by adding filter option to modify "subscription" games (Galaxy restart needed, no live changes possible)

OR

by allowing to return not chosen, potential games (type 1) from "owned games" AND mock them to have unrevealed keys.

I like the first option more, as second would have to do some messy hacks with game types.

nosyn00b commented 3 years ago

Could't you simply ignore (not passing the single games to GOG) if the the game is in status 3 and "show reveled keys" is not flagged?

nosyn00b commented 3 years ago

Considering the addition in the FAQ section, this specific issue can be closed, if you agree. While "hide choices with already revealed keys" considerations could be raised as a different thread/issue, but I think there is already another one somewhere!

UncleGoogle commented 3 years ago

Could't you simply ignore (not passing the single games to GOG) if the the game is in status 3 and "show reveled keys" is not flagged?

I do that already in get_owned_games interface. The game is still visible in Galaxy because it is returned from get_subscription_games interface.

In the second interface I have no information about keys state.

But yes, those interfaces could theoretically depend on each other using cached games from one of them, something like:

def get_subscription_games(self, subscription_name):
    sub_games = ...

    if not self._settings.show_revealed_keys:
        revealed_key_games = [g.machine_name for g in self._owned_games if isinstance(g, Key) and g.key_val]
        yield [game in sub_games if game.machine_name not in revealed_key_games]
    else
        yield sub_games

that's one idea. I don't remember why I didn't want to go that direction, need to recap potencial drawbacks.. otherwise that's briliant and can be used to fix #123

EDIT

one obstacle is that in current implementation you have to have checked "keys" and not checked "show_revealed_keys".

If you unselect "keys" self._owned_games does not contain keys at all.

UncleGoogle commented 3 years ago

Considering the addition in the FAQ section, this specific issue can be closed, if you agree. While "hide choices with already revealed keys" considerations could be raised as a different thread/issue, but I think there is already another one somewhere!

right, let me close this one.

nosyn00b commented 3 years ago

In the second interface I have no information about keys state.

perhaps you can understand status of the games looking for it in the events collection