Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When you start the client in offline mode unstarring still works #935

Closed
sssoleileraaa opened this issue Mar 13, 2020 · 9 comments · Fixed by #936
Closed

When you start the client in offline mode unstarring still works #935

sssoleileraaa opened this issue Mar 13, 2020 · 9 comments · Fixed by #936
Labels
bug Something isn't working

Comments

@sssoleileraaa
Copy link
Contributor

sssoleileraaa commented Mar 13, 2020

STR

  1. Make sure you have a starred source
  2. Log into the client in offline mode
  3. While offline, at any point during your session, click on the star to unstar the starred source

Expected

The "must sign in" message and for starring to not work

Actual

The "must sign in" message but starring does work

Note: This was happening on master before #933 was merged

@sssoleileraaa sssoleileraaa added bug Something isn't working needs/reproducing labels Mar 13, 2020
@rmol
Copy link
Contributor

rmol commented Mar 13, 2020

I think this is going to be fixed by #916.

@sssoleileraaa
Copy link
Contributor Author

Looks like it is not fixed by #916 sadly :(

@redshiftzero
Copy link
Contributor

I just pulled latest master (7066e9b) and for me behavior is as expected both when I start the client in offline mode and I start the client in online mode and then logout

@eloquence
Copy link
Member

eloquence commented Mar 13, 2020

I am also not able to reproduce this in a 0.2.3-deb prod build. I consistently get the "You must sign in" error.

@sssoleileraaa
Copy link
Contributor Author

Oops I just realized that I get the "You must sign in" error but starred sources can still be unstarred. Lemme show a gif...

@sssoleileraaa
Copy link
Contributor Author

Peek 2020-03-13 15-42

@eloquence
Copy link
Member

Not able to see this in the prod build, but let me try in master

@eloquence
Copy link
Member

eloquence commented Mar 13, 2020

Yup, see the buggy behavior in master in Qubes.

@eloquence
Copy link
Member

I can reproduce the broken behavior on commit b231971 (part of #872), but not before it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants