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

classic update for 2.0.1 #708

Open
wants to merge 778 commits into
base: classic
Choose a base branch
from
Open

classic update for 2.0.1 #708

wants to merge 778 commits into from

Conversation

yuvaraj119
Copy link

No description provided.

Grigory Fedorov and others added 30 commits April 24, 2015 18:23
…oved to drawer from options menu. New icons.
Set textColor value to textColorPrimary attribute to facilitate theme'ing
Conflicts:
	app/src/main/res/xml/preference_editor.xml
Grigory Fedorov and others added 30 commits July 3, 2015 10:54
XEP Status Code 100 tells the XMPP client application, that a room is not
anonymous, which means that JIDs of all members are revealed.

Unfortunately, XEP states, that the Server should send an body together
with the status code, containing 'This room is not anonymous'.

Extract from XEP 1.24:
"Example 24. Service Warns New Occupant About Lack of Anonymity

<message
    from='[email protected]'
    to='[email protected]/pda'
    type='groupchat'>
  <body>This room is not anonymous.</body>
  <x xmlns='http://jabber.org/protocol/muc#user'>
    <status code='100'/>
  </x>
</message>"

Therefore each time a user enters a non-anonymous room he receives a
message containing "This room is not anonymous".
A problem with this is the fact, that this happens with every reconnect
of the mobile connection. For each room with every internet change (WiFi
to mobile or vice versa, or just unstable internet) one'll get a
notificaton.

To stop this from happening I added an Option to the Chat-Room-Settings,
which allows to suppress this messages. This way users can suppress them
for the rooms they don't need this notificaton.
…se (Portugal) and Russian translations updated.
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.

9 participants