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
One participant message #4528
One participant message #4528
Conversation
…s joined the conference.
Hi, thanks for your contribution! |
# Conflicts: # react/features/base/participants/components/ParticipantView.native.js
I have signed CLA. |
Hi there! First, thanks for walking the extra mile here and making the PR work for both iOS and Android from the get go. While I entertain the idea behind this patch, I'm not very fond of the implementation. We generally don't want to get text directly from props and render it, because it wouldn't be internationalized. So here is a proposal!
This way the text can be translated and every app doesn't need to do it on their own, should they want to use this feature. |
@saghul
Let me know if anything else you want to modify/add. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Sorry I drropped the ball on this one :-/ I took another look and AFAIS the native parts still take the string. Now that the flag is implemented you should be able to drop all that code. |
@saghul |
@patidars That is not something I'm keen on merging since it bypasses all forms of translation. If you want different messages you can alwaysmodify main.json and make a custom build. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Added a way to display message like “Please wait for others to join”.
Exposed an optional props from mobile native platform called ‘onlyOneParticipantMessage’. If we pass this prop from native(android/iOS) and there is only one participant in conference then this message will get displayed. If there are more than one participant in conference then this message will not get display.
This could be a welcome message which can tell user to be on conference and wait until others join.