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

Marking room as “Favourite” is not working #3309

Closed
pReya opened this issue Sep 20, 2024 · 9 comments
Closed

Marking room as “Favourite” is not working #3309

pReya opened this issue Sep 20, 2024 · 9 comments
Labels
A-Room-Detail A-Room-List O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue

Comments

@pReya
Copy link

pReya commented Sep 20, 2024

Steps to reproduce

  1. Go into any rooms settings
  2. Activate the “Favourite” toggle
  3. Go back to room view or room list
  4. Go back into the room settings
  5. See: “Favourite” state is back to “off”/default state again

Outcome

The “Favourite” status should be persisted/applied/saved

Your phone model

iPhone 13 Mini

Operating system version

18.0

Application version

1.8.1

Homeserver

Synapse 1.115

Will you send logs?

Yes

@pReya pReya added the T-Defect label Sep 20, 2024
@pixlwave pixlwave added X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue A-Room-List S-Major Severely degrades major functionality or product features, with no satisfactory workaround O-Occasional Affects or can be seen by some users regularly or most users rarely A-Room-Detail labels Sep 26, 2024
@stefanceriu
Copy link
Member

I can no longer reproduce this as of 1.8.5 (unreleased). Closing it for now but do feel free to reopen/comment if you still have issues.

@pReya
Copy link
Author

pReya commented Sep 27, 2024

Still present in 1.8.3

Will wait for 1.8.5 release to confirm if it was fixed or not.

@pReya
Copy link
Author

pReya commented Oct 19, 2024

I can no longer reproduce this as of 1.8.5 (unreleased). Closing it for now but do feel free to reopen/comment if you still have issues.

Still present in 1.9.1. Can you reopen the ticket? @stefanceriu

@stefanceriu
Copy link
Member

I still can't reproduce it, is there anything special about your stack?

@pReya
Copy link
Author

pReya commented Oct 21, 2024

I still can't reproduce it, is there anything special about your stack?

No, nothing special. Synapse v1.115.0 with pretty much default settings. I can also reproduce it on a second account/phone (on the same server). So this is not specific to my account or phone. I can also see it working using a matrix.org account. So this has to be somehow server-specifc. I'll try to use some other servers, too, to see how many of them are affected. I'd be happy to share screen recording/logs/configs privately.

@stefanceriu
Copy link
Member

It's supposed to be stored as a tag under the room account data e.g.

{
  "type": "m.tag",
  "content": {
    "tags": {
      "m.favourite": {}
    }
  }
}

There might also be some errors in the logs if you can submit a rageshake, or copy paste those lines here (settings -> report a problem -> view logs)

@marwing
Copy link

marwing commented Oct 21, 2024

This is a missing feature in synapse that was first implemented in v1.116.0 (element-hq/synapse#17694, element-hq/synapse#17707).

@stefanceriu
Copy link
Member

Ah there we go, nice one marwing! 👏

@pReya
Copy link
Author

pReya commented Oct 21, 2024

This is a missing feature in synapse that was first implemented in v1.116.0 (element-hq/synapse#17694, element-hq/synapse#17707).

Thanks for the info! Can confirm, it's working now that I've upgraded to v1.117.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Room-Detail A-Room-List O-Occasional Affects or can be seen by some users regularly or most users rarely S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Needs-Rust This issue needs a Rust SDK change. It must have a link to a Rust SDK issue
Projects
None yet
Development

No branches or pull requests

4 participants