First of all djmattyg007 thanks again for all the comments.
I made many improvements and changes to the script. Some are a result of your comments, I added detailed answers to the comments you raised.
The script is now available on the google code space, but only as a package. I plan to get through another session of bug testing before releasing it to the xbmc-addons SVN.
I'll post another reply with a change log tomorrow on this thread.
djmattyg007 Wrote:There are several problems with this script. For starters, I couldn't get it working at all with the XBMC for Windows.
That's funny since I'm working on Windows, so I would expect it won't be the problematic platform
. I understand that you did manage to get it working, can you please share more details?
djmattyg007 Wrote:On the Xbox, you are obviously unable to use OAuth. I had to manually edit the settings.xml file to get the password field to show up. And even after authenticating it properly for the first time, I had no idea this had occurred because no message was displayed on-screen at all. I had to go into the debug log and check for myself.
As mentioned, XBOX doesn't offer any browser capabilities, a must for OAuth. Therefore, the only option is to use the username/password option. I fixed the settings issue and the field is now visible. Regarding the informational display of authentication success/failure, you have a valid point here and I'll add more visual indications of the process.
djmattyg007 Wrote:Because I use the default Project Mayhem III skin, I do not have skin integration, and have no way of just pushing a button to tweet about individual songs or videos. The lack of an interface for this script is rather annoying.
I wish to avoid putting an "real" UI, I'm lousy at this and I find myself wasting time on a "pixel here, pixel there" issues which drives me nuts
.
So, because you don't have the option to use a dedicated button of the skin, I added a proper support for keyboard/controller shortcut (keymap.xml or the new keyboard.xml). It works very well and you can tweet away with a click of a button. A question back to you, what I did is, when you click on the shortcut it will pop-up a pre-populated keyboard screen that you approve and it tweets away. The field is pre-populated based on your custom tweets and fully editable. Is this what you had in mind?
First of all djmattyg007 thanks again for all the comments. If anyone else downloaded the script, please provide your feedback, it's very helpful.
djmattyg007 Wrote:There are several problems with this script. For starters, I couldn't get it working at all with the XBMC for Windows.
That's funny since I'm working on Windows and I don't have the chance to test it on other platforms
. I understand that you did manage to get it working, can you please share more details?
djmattyg007 Wrote:On the Xbox, you are obviously unable to use OAuth. I had to manually edit the settings.xml file to get the password field to show up. And even after authenticating it properly for the first time, I had no idea this had occurred because no message was displayed on-screen at all. I had to go into the debug log and check for myself.
As mentioned, XBOX doesn't offer any browser capabilities, a must for OAuth. Therefore, the only option is to use the username/password option. I fixed the settings issue and the field is now visible. Regarding the informational display of authentication success/failure, you have a valid point here and I'll add more visual indications of the process.
djmattyg007 Wrote:When the automatic tweeting options are on, xbTweet pays no attention to how long the file has been playing for. It automatically tweeted everything I played as soon as it started playing.
Is this based on the debug log? If so, then it's just a debug print, there's no actual tweet. If it's based on the percentage of play, this is a bug. Can you provide more information?
djmattyg007 Wrote:Also, I am not sure how much control you have over this, but when not using OAuth, tweets aren't shown to be coming from "xbTweet", just "API". For me, this is annoying beyond belief and probably means I won't be using it.
ANNOYING is the word! Sorry mate, can't control it. Only OAuth offers an application name in the tweet. Maybe someone knows a different way, but that's what I know.
djmattyg007 Wrote:Another small complaint I have is the number of hashtags you insist on including in the default messages. It's completely unnecessary to include #xbTweet in the tweet when it already says "from xbTweet" right underneath it.
I separated the tags from the custom tweet in the settings page, this allows you now to have full control over what is tweeted. BTW, you always had full control, the default is with #xbTwitter. I thought it would be nice to see the usage of the script during its early days. That's why I kept it as part of the default setting.
djmattyg007 Wrote:The option to automatically follow the creator should be deselected by default in the settings, or simply removed altogether. I should have full control over who I'm following and something like that is relatively misleading.
Very valid comment! Since the code is now released in the Repo SVN and users don't have a way to read a disclaimer or any kind of notice informing them in advance about it, I changed the setting to be 'no' as default.
djmattyg007 Wrote:Upon inspection of the code in this project I've also found that there are still references to "xbTwitter", despite the project being renamed to "xbTweet" as per Twitter's rules on what API clients can be named. You should probably get around to fixing this ASAP.
Thanks, fixed.
If anyone else downloaded the script, please provide your feedback, it's very helpful.