Skip to content
This repository has been archived by the owner on Jun 7, 2020. It is now read-only.

[BUG] Room stays hidden #2158

Open
soundstorm opened this issue Sep 1, 2018 · 10 comments · May be fixed by #2566
Open

[BUG] Room stays hidden #2158

soundstorm opened this issue Sep 1, 2018 · 10 comments · May be fixed by #2566
Labels

Comments

@soundstorm
Copy link
Contributor

If you hide a room, search for it and open it, the room should be shown again.
A refresh of the list is just performed after reopening (switching back from another app or killing and reopening) Rocket.Chat.

@Sameesunkaria
Copy link
Collaborator

Hmmm. This is working okay on my phone. 🤔

@soundstorm
Copy link
Contributor Author

@Sameesunkaria try to hide a faved room :)

@soundstorm
Copy link
Contributor Author

Okay, seems like an edge case. Sometimes it refreshes fast enough, sometimes not 🤨

@Sameesunkaria
Copy link
Collaborator

Hiding a favorited room works as well. Seems like it must be an edge case.

@Sameesunkaria
Copy link
Collaborator

Experienced the glitch on an iPad.

@tmbiOS
Copy link

tmbiOS commented Feb 20, 2019

@rafaelks Unhide channel after searching doesn't work. (not fetching API - open groups/channels)
Version 3.3.1
I can do it and create merge request

@tmbiOS
Copy link

tmbiOS commented Feb 20, 2019

@rafaelks Rafael, I'm trying to add a new feature - unhide channel.
I created API, but I don't understand how to reload SubscriptionsViewController.
It works only after restarting the app.
Can you advice how to fix it?

@rafaelks
Copy link
Contributor

@tmbiOS Realm should reload the list, are you applying the changes to Realm?

@tmbiOS
Copy link

tmbiOS commented Feb 25, 2019

@rafaelks
I created websocket API (or REST API), than i added:

Realm.executeOnMainThread({ _ in
      subscription.open = open
    })

Should I something like this?

let subscription = Subscription.getOrCreate(realm: realm, values: values, updates: { object in
              object?.open = object?.open
            })

            realm.add(subscription, update: true)

This doesn't work.

@rafaelks
Copy link
Contributor

@tmbiOS Can you open a PR with your changes?

@tmbiOS tmbiOS linked a pull request Feb 26, 2019 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants