[jitsi-dev] XMPP Message correction


#1

Hi,

I'm currently adding Message correction to Jitsi according to XEP-308. I
understood you have it implemented in Swift so I decided to test if they
are compatible.

I saw in your logs (btw adding a debugging console to the UI does make
testing faster :slight_smile: ) that while you do use the <replace /> tag and the 'id'
attribute as expected, you're using a different namespace - namely, '
http://swift.im/protocol/replace'. Is it OK for you to change this to the
NS listed in the XEP?

Thanks,
Ivan


#2

Hey Kevin,

Thanks for the info. I thought that you might've fixed it in some newer
release and checked that I have the last one, but I didn't get to building
from sources. Good to hear the XEP ns is supported now.

Ivan

路路路

On Wed, Jun 27, 2012 at 9:56 AM, Kevin Smith <kevin@kismith.co.uk> wrote:

On Tue, Jun 26, 2012 at 11:18 PM, Ivan Vergiliev > <ivan.vergiliev@gmail.com> wrote:
> Hi,
>
> I'm currently adding Message correction to Jitsi according to XEP-308. I
> understood you have it implemented in Swift so I decided to test if they
are
> compatible.
>
> I saw in your logs (btw adding a debugging console to the UI does make
> testing faster :slight_smile: ) that while you do use the <replace /> tag and the
'id'
> attribute as expected, you're using a different namespace - namely,
> 'http://swift.im/protocol/replace'. Is it OK for you to change this to
the
> NS listed in the XEP?

We had an implementation long before I published the XEP and I forgot
to update the namespace for a while, but it should be sending the
correct namespace since

http://swift.im/git/swift/commit/?id=08399fe0e7d8d5ab42975b51c487c82caf845cef

/K


#3

Thanks, that would be helpful. I'm using Mac OS 10.7.

Ivan

路路路

On Wed, Jun 27, 2012 at 1:15 PM, Kevin Smith <kevin@kismith.co.uk> wrote:

On Wed, Jun 27, 2012 at 11:14 AM, Ivan Vergiliev > <ivan.vergiliev@gmail.com> wrote:
> Thanks for the info. I thought that you might've fixed it in some newer
> release and checked that I have the last one, but I didn't get to
building
> from sources. Good to hear the XEP ns is supported now.

If you're on Windows, Mac, Ubuntu or Debian I can send you an updated
package to save you building from source.

/K