search

realm / realm-dotnet

Realm is a mobile database: a replacement for SQLite & ORMs
https://realm.io
Apache License 2.0
1.25k stars 164 forks source link

App.AllUsers return include logged out ? #3650

Closed carbonete closed 3 months ago

carbonete commented 3 months ago

What happened?

Hi in my doubt is what App.AllUsers return because in .NET SDK release 12.1.0 notes say

App.AllUsers included logged out users only if they were logged out while the App instance existed. It now always includes all logged out users. (Core 14.6.0) ( Release 12.1.0 · realm/realm-dotnet · GitHub )

But in Realm .NET SDK Docs say :

AllUsers Gets all currently logged in users. ( Class App | Realm )

What is correct , i use it in my app and until now i consider only user logged in will returned.

Thanks

Repro steps

visit urls

Version

12.1.0

What Atlas Services are you using?

Atlas App Services: Functions or GraphQL or DataAPI etc

What type of application is this?

Xamarin

Client OS and version

Android 10

Code snippets

No response

Stacktrace of the exception/crash you're getting

No response

Relevant log output

No response

sync-by-unito[bot] commented 3 months ago

➤ PM Bot commented:

Jira ticket: RNET-1167

papafe commented 3 months ago

Hi @carbonete, thanks for reporting this. What you read in the release should be correct, so App.AllUsers should contain both logged in and logged out users. We need to fix the docs of the API

carbonete commented 3 months ago

@papafe Thanks

Please see this page https://www.mongodb.com/docs/atlas/device-sdks/sdk/dotnet/logging/

I think after 12.3.0 release this need revision too.