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

[doc] Things I'd like to see documented #342

Closed
jfarcand opened this issue May 4, 2012 · 4 comments
Closed

[doc] Things I'd like to see documented #342

jfarcand opened this issue May 4, 2012 · 4 comments

Comments

@jfarcand
Copy link
Member

jfarcand commented May 4, 2012

See

   http://groups.google.com/group/atmosphere-framework/msg/0856e98d2bbbd816
@chris-martin
Copy link

Why would I want or need to set a particular atmosphere property to 'true' or 'false'"?

This remark perfectly summarizes my frustration with the documentation. AtmosphereResource, in particular, has some fundamental concepts that could use some explanation. The following kind of thing is very unhelpful:

     * @return <tt>true<//tt> if that {@link AtmosphereResource#resume()} has been
     *         invoked and set to <tt>true</tt>
     */
    public boolean isResuming();

The biggest problem is that it leaves unanswered: Under what circumstance does the Atmosphere framework invoke #resume()? I still don't fully understand what it means for a resource to "be resuming". Same goes for the rest of the state boolean on Atmosphere resource (suspended, resumedOnTimeout, cancelled). I'm sure these properties are probably related to each other in some way. For example, can a resource be both suspended and cancelled at the same time? I would think not, but I'm not sure.

@jfarcand
Copy link
Member Author

jfarcand commented May 8, 2012

@chris-martin I fully agree. Feel free to file more issue with such non sense comments I wrote. For sure I will work on that. Thanks!

@jfarcand
Copy link
Member Author

Added a table for Atmosphere Configuration attributes

https://github.com/Atmosphere/atmosphere/wiki/Configuring-Atmosphere-using-the-web.xml-or-atmosphere.xml

@jfarcand
Copy link
Member Author

jfarcand commented Jun 8, 2012

Mark this bug as fixed. Please file new one with specific requests if you think the current wiki lack.

@jfarcand jfarcand closed this as completed Jun 8, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants