Hi Dave (and any others),
I think the Documentation is mostly complete (enough for beta?). Please check it over for accuracy and clarity, and provide feedback (more critical the better). You've been answering most of the issues, perhaps you could help me expand the common issues section, and add in some bits about the current limitations, and perhaps what the goals are.
Cheers.

Comments

nlowhor’s picture

nlowhor commented

Where can I find the documentation? I'm still a bit of a new user, but I've been spending a lot of time trying to get an app working lately. I'd be happy to read it and see what I can offer.

frankcarey’s picture

frankcarey commented

http://drupal.org/node/195035.

... there is now a link on the project page...

I tried to make it as step by step as possible, so Its good you are a new user, you cab provide good feedback about the documentation. suggestions and constructive criticism welcome :)

nlowhor’s picture

nlowhor commented

Ok, I'm going to give it a read and just jot notes as I think of things.

1) Installation step 3 - for some reason I couldn't get the require to work, I just copied everything from the settings.inc into my settings.php file and it worked
2) The section for the API Key has no heading. The text for the Label just flows right into the text about the API key.
3) I think more detailed explanation is needed for the Facebook navigation and user settings sections. I still find a lot of this confusing although it's certainly making more sense to me now. One thing that I'm still confused about is there seem to be sections that indicate that you could create an app that requires a login to the app? It's just confusing when you have a login for facebook, the action of adding the app which may or may not add a drupal account, and you have a drupal login. There's a lot going on there and I think some of the issues I'm currently having with my app might be related to how I have this setup.
4) You refer to instructions for setting up infinite sessions but I didn't see anything about that. It can certainly be a bit difficult.
5) You might look through the issues and support requests for anything I have posted. It might give you some ideas for other common issues as I've posted a few things I bumped into.

Otherwise, I think this looks great! Nice work. Surely would've been helpful a few weeks ago when I started looking at this stuff. :)

frankcarey’s picture

frankcarey commented

Thanks nlowhor , just the info i need.

1) Require may be not pulling out of the right directory , maybe wrong path?, maybe a "/" at the beginning include's path?

require_once "modules/fb/settings.inc" ?

Thanks again, Cheers

nlowhor’s picture

nlowhor commented

No problem. Let me know if I can help further. Thanks....I got that require_once working now :)

Dave Cohen’s picture

Dave Cohen commented

Frank,

I looked over that page and made some edits. Thanks again for getting the ball rolling.

I'm hesitant to call anything a "beta" yet. I still feel like major features and data are in flux, and I want people to realize this before relying on the modules.

Eventually, the doc should be expanded, and broken up into multiple pages. I know it's going to take some time and effort...

frankcarey’s picture

frankcarey commented

No worries, thanks for the edits. Didn't mean to imply module was beta ready, just thinking the documentation would be required for beta.
Cheers

nk126’s picture

nk126 commented

Hey, this is great. Trying to give a little back for the work Dave's done on the module, and so read over the documentation looking for edits ... I'm wondering if this statement is 100% true:

"It is not necessary to enable the fb_fbml theme. See the installation instructions above."

I had to enable the theme to get pages to render properly under the fb_fbml theme. With the theme installed in the right place but not enabled, they were showing up on facebook under my default theme, which was no good at all (we did a lot of customization at the page.tpl level). Only after I actually enabled fb_fbml via the admin pages on my site did my facebook app pages start rendering with fb_fbml.

Maybe it's some kind of anomaly on my install, or maybe you meant to imply that "your app will still work in facebook if you don't enable the fb_fbml theme, but you need to enable the theme in order to take advantage of its theming" ?

I also had trouble initially with the require_once, so I'm glad to see it documented here. Actually, nlowhor wrote much of what I would have said myself! I can't say much about the user settings and infinite sessions part of the documentation b/c I don't fully understand it yet myself ... But hopefully I will soon enough.

Frank, I'm happy to help expand this documentation as per Dave's suggestion. If you want the help let me know. I hope to be using this module a lot in the new year (hope to be doing a fair amount of FB development in general), so I'd like to give back what I can to the Drupal community.

nk

frankcarey’s picture

frankcarey commented

Thanks, feel free to get doc team status and make edits... though we should have like a wiki "discussion page" somewhere... maybe just keep this thread going? Instructions on getting edit permissions are at the bottom of the doc page.

As for having to enable fbml theme.. I reacall Dave saying something about having to go to the theme page to get the theme in the database, but that you don't actually have to enable it... i'm asking him about the details of this as well.

Cheers

Dave Cohen’s picture

Dave Cohen commented

I've never had to enable the theme myself. It relies on a global variable called $custom_theme for this to work. Is it possible you have a module installed that overrides that?

lape’s picture

lape commented
Status: Needs review » Fixed

Is this still an issue?

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.