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

Update Realm to 3.7.1 #58

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AleksanderMielczarek
Copy link

@AleksanderMielczarek AleksanderMielczarek commented Sep 9, 2017

Hello,

unfortunately stetho-realm does not work with latest Realm 3.7.1. This PR changes LinkView with OsList and add OsRealmConfig.Durability (this is also covered in #57).

@guness
Copy link

guness commented Sep 21, 2017

if this works well, please merge.

@Zhuinden
Copy link

Zhuinden commented Sep 21, 2017

I also found that

        sb.append(linkList.getTargetRowIndex(pos));

should most likely be equivalent to

        sb.append(linkList.getUncheckedRow(pos).getIndex());

@zaki50 any chance?

@mgohin
Copy link

mgohin commented Sep 28, 2017

Realm version is now 3.7.2 :)

Copy link

@mak0t0san mak0t0san left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried this out Realm 3.7.2 and it worked! My android app no longer crashes when inspecting and I am able to browse the data.

Copy link

@mak0t0san mak0t0san left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried this out Realm 3.7.2 and it worked! My android app no longer crashes when inspecting and I am able to browse the data.

@AleksanderMielczarek
Copy link
Author

Any chance to merge this one?

@zaki50
Copy link
Member

zaki50 commented Sep 29, 2017

@AleksanderMielczarek Thank you for making this PR.
I think that we need compatibility with older versions. I'll try to integrate your change with my local work in this weekend.

Could you give me a bit of time?

@Benyuss
Copy link

Benyuss commented Oct 16, 2017

Hi @zaki50 !

Any updates on this please?

@eygraber
Copy link

Realm 4.1 was released.

@bloderxd
Copy link

How we're going with that?

@pavel-mukhanov
Copy link

Maybe we should fork this repo and assign new maintainer?

@eygraber
Copy link

Realm 4.3.1 was released

@Zhuinden
Copy link

I guess it might be possible to use Realm Studio to check one's realm schema and data and all that, so Stetho-Realm just isn't that useful anymore?

@eygraber
Copy link

@Zhuinden isn't there a whole process for apps that don't use sync, and aren't rooted? Something like putting the realm file in public storage for Dev builds, and then adb-ing it to the dev computer?

@Zhuinden
Copy link

Zhuinden commented Dec 14, 2017

@eygraber I think now with AS 3.0+ you can just use Device File Explorer and use "save as" on the file to save it on local pc

@eygraber
Copy link

But it still needs to be on public storage, right?

When I have time, I'd love to do some work on https://github.com/jonasrottmann/realm-browser

Seems like a much more flexible solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.