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

new demo: my word #60

Merged
merged 6 commits into from
Feb 27, 2013
Merged

new demo: my word #60

merged 6 commits into from
Feb 27, 2013

Conversation

hzhua
Copy link
Contributor

@hzhua hzhua commented Feb 25, 2013

I only test this demo in sina weibo.

@hupili
Copy link
Owner

hupili commented Feb 26, 2013

@xuanqinanhai

Can you help to test this?

Many thanks.

I'm running out of time slots. If it is tested OK for both of you, I'll merge it.

@hzhua

You can also ask people in the group to test the app.

@daimajia
Copy link

QQ20130226-1

it works well ! @hupili

@hupili
Copy link
Owner

hupili commented Feb 26, 2013

How about another platform, e.g. renren?

@daimajia
Copy link

QQ20130226-2

I tested... it failed

The reason should be the different structure between the SinaWeiboStatusMessage and RenrenStatusMessage

@hupili

@hupili
Copy link
Owner

hupili commented Feb 27, 2013

I see...

@hzhua

When you de-duplicate messages by signature, can you exclude 'ID' from the string? Using fields in 'parsed' usually suffices. If you must use ID (e.g. same user ask same word at the same platform at the same time... ), please use ID as a whole. MessageID object can be str()-ed as a whole. Please do not tap into the fields of MessageID, where those fields are mainly for internal use of SNSAPI methods and they are different across platforms.

Similarly, when you refer to other objects in this project, please do not use non-mandatory fields if possible. Since data types are heterogeneous, we only guarantee mandatory fields are there. In this way, you can make your app generalize to all platforms without worrying what is that platform (the purpose of SNSAPI).

I added some description in the doc string.

http://snsapi.ie.cuhk.edu.hk/doc/snsapi.html#module-snsapi.snstype

@hzhua
Copy link
Contributor Author

hzhua commented Feb 27, 2013

I have replace 's.ID.id' with 's.ID'.

@xuanqinanhai

Please help test it on renren. I can't authorized from renren.
Many thanks!

for s in status_list:
s.show()
msg_string = "".join( unicode(x) for x in \
[s.parsed.time, s.ID, s.parsed.username, \
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Use s.ID instead of s.ID.id

@daimajia
Copy link

QQ20130227-3

it works well in Renren...

QQ20130227-4

however it's a little weird that the status is commented by the users account.

thanks for your hard work.

@hupili
Copy link
Owner

hupili commented Feb 27, 2013

... Are you running this app under account "@林惠文" or "@this_is_test"?

@daimajia
Copy link

I used my renren account 林惠文
and
the status is @this_is_test miracle

@hupili
Copy link
Owner

hupili commented Feb 27, 2013

I see. That explains the screenshot.

Once somebody deploys this app under a "@dict" account, others can add ask trans by posting status.

The last piece is the notification platform. Once ready, it should be able to handle notification seamlessly.

@daimajia
Copy link

we can use xiaohuangji's notification handler system directly. filter the @ information,and then reply .

right?

@hupili
Copy link
Owner

hupili commented Feb 27, 2013

@xuanqinanhai

yes. I was wrapping xiaohuangji to SNSAPI Language last Sunday. Half done.

hupili added a commit that referenced this pull request Feb 27, 2013
new demo: word translation via home_timeline and reply
@hupili hupili merged commit 33d9db3 into hupili:dev Feb 27, 2013
@hzhua hzhua deleted the word_demo2 branch February 27, 2013 04:29
@daimajia
Copy link

looking forward to it.

and hope to give me some time to finish github's pull request. little tired recently.

@hupili
Copy link
Owner

hupili commented Feb 27, 2013

Take your time~

I'm also in all kinds of rush recently...

@daimajia
Copy link

mate , take care of your body and health.

these are the most important things than others.

@hzhua hzhua restored the word_demo2 branch February 27, 2013 05:20
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.

3 participants