question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. ItĀ collects links to all the places you might be looking at while hunting down a tough bug.

And, if youā€™re still stuck at the end, weā€™re happy to hop on a call to see how we can help out.

[Android] [SDK 2.0.0] Feedback when integrating new SDK to Riot.im application

See original GitHub issue

Hi,

Iā€™m integrating Jitsi 2.0.0 to Riot. Conference call is well setup and working fine šŸ‘

My feedbacks so far:

  • Is there a way to set the title of the screen? I have the pb reported (and fixed) here #4050, but there is no API (yet?) in JitsiMeetConferenceOptions to set the title
  • currently we use a JitsiView, and there is a new method leave(). We are using dispose() for now, should I replace the call with leave()?
  • embedded chat is not working well: text and button is hidden behind the keyboard. Also the chat is not displayed above the video as before.
  • in the menu, clicking on ā€œMeeting infoā€ share the URL of the conf, which looks more like action ā€œInvite someoneā€, but maybe I do not understanf it correctly. ā€œInvite someoneā€ action is disabled.

These are not blocking issues, so I can push Riot to the prod with this SDK.

Thanks!

[Edit] Also, when uploading Riot to the PlayStore it warned me that new permissions where added: android.permission.READ_CALENDAR and android.permission.WRITE_CALENDAR (Added from the react-native-calendar-events:1.6.4-jitsi-799011 manifest).

Issue Analytics

  • State:closed
  • Created 4 years ago
  • Comments:6 (3 by maintainers)

github_iconTop GitHub Comments

1reaction
saghulcommented, Apr 6, 2019

About the title, why not add a setTitle() method in the JitsiMeetConferenceOptions to configure the bundle as per #4050? This could be a temporally workaround to avoid having a conf id displayed.

Yeah, I think that makes sense, since the local parameter would take precence over the room setting, but I need to double-check.

Also can i remove the calendar permissions safely, i mean does jitsi will handle the lake of calendar permissions? In riot we do not display the welcome page

Yep, you can. We donā€™t trigger any permissions prompt since that only happens on user action on the welcome page.

1reaction
saghulcommented, Apr 5, 2019
* Is there a way to set the title of the screen? I have the pb reported (and fixed) here #4050, but there is no API (yet?) in JitsiMeetConferenceOptions to set the title

There is no way to set the title as of yet. We need to see how this blends together with the conference subject. We have this concept (the XMPP MUC subject basically) but havenā€™t exposed it to the UI yet.

* currently we use a JitsiView, and there is a new method `leave()`. We are using `dispose()` for now, should I replace the call with `leave()`?

First, Iā€™d recomend you switch to using the JitsiMeetActivity directly, unless you have some very very specific needs. As for leave vs dispose, you can keep on calling dispose since it does a leave internally.

* embedded chat is not working well: text and button is hidden behind the keyboard. Also the chat is not displayed above the video as before.

Iā€™m not seeing this in Jitsi Meet (the app). Can you post a screenshot?

* in the menu, clicking on "Meeting info" share the URL of the conf, which looks more like action "Invite someone", but maybe I do not understanf it correctly. "Invite someone" action is disabled.

These are 2 different things, related, which we need to refine a bit. Inviting someone requires a backend service which processes invites / dials out. Meeting info contains the meeting information just like before, with the system share sheet.

Also, when uploading Riot to the PlayStore it warned me that new permissions where added: android.permission.READ_CALENDAR and android.permission.WRITE_CALENDAR (Added from the react-native-calendar-events:1.6.4-jitsi-799011 manifest).

Thatā€™s correct, there is calendar integration now. If calendar access is provided, meeting notifications will be shown in the conference, and for those using the welcome page, upcoming meetings show up there.

Read more comments on GitHub >

github_iconTop Results From Across the Web

riots.im
This site hosts historical Riot Web releases for testing purposes. Releases after version 0.8.3 are pulled from GitHub into Amazon S3 and served...
Read more >
NextGen SDK | Android Integration | Smaato
Load a native ad. Render a native ad in NativeAd.Listener.onAdLoaded; Lastly, build and run the app. The native ads view should appear on...
Read more >
matrix-react-sdk | Yarn - Package Manager
This is a react-based SDK for inserting a Matrix chat/voip client into a web page. This package provides the React components needed to...
Read more >
Synapse 1.10.0 released - Matrix.org
Get the new release from github or any of the sources mentioned at ... Riot-Android (yes, the legacy app!): We have fixed the...
Read more >
Untitled
#status Koalition kurze definition, Nitech srl cluj, New baby freebies, Marea desarraigo? ... Android app on google play logo, La gata salvaje capitulo...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found