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.

Sign up for GitHub

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

Jump to bottom

Room class #65

Closed
banksJeremy opened this issue May 14, 2014 · 4 comments
Closed

Room class #65

banksJeremy opened this issue May 14, 2014 · 4 comments
Assignees
@banksJeremy
Labels
enhancement

Comments

@banksJeremy
Copy link
Collaborator

as with the Message class #51 and User class #52
@banksJeremy banksJeremy self-assigned this May 14, 2014
banksJeremy pushed a commit that referenced this issue May 14, 2014
@jeremyBanks
Adds incomplete User and Room, and a failing test setting some goals.
a572d36 
        me = client.get_me()
        sandbox = client.get_room(11540)

        with sandbox.messages() as messages:
            sandbox.send_message("hello worl")

            for message in messages:
                if message.owner is me:
                    assert my_message.content == "hello worl"

ref #65, ref #52, ref #43
banksJeremy pushed a commit that referenced this issue May 15, 2014
@jeremyBanks
Starts using Room and User instances instead of bare names and IDs.
e077b09 
They still have no scraping implemented for unknown values.

ref #52, #65
@banksJeremy banksJeremy mentioned this issue May 15, 2014
Merged
6 tasks
banksJeremy pushed a commit that referenced this issue May 15, 2014
@jeremyBanks
Adds User profile scraping and room info scraping, with simple tests.
826cd98 
Moves some logic from client to room.

Adds simple docstrings with epydoc annotations to public fields and
methods of Client. Also adds epydoc as a dev dependency, and adds
`make epydocs`.

It probably isn't necessary to add this documentation to our internal
methods, and I won't bother with browser for now, but I think it's
useful to have for our user-friendly interface. (Both for users directly
and through IDEs that recognize the type annotations, like PyCharm CE.)

resolves #52 - let's forget about slugs and message sets for now. We can
reply to messages and a weak set of current messages might be pointless.

ref #65, #43 - event/message iterators/contexts are still outstanding.
banksJeremy pushed a commit that referenced this issue May 15, 2014
@jeremyBanks
Adds User profile scraping and room info scraping, with simple tests.
6f213cc 
Moves some logic from client to room.

Adds simple docstrings with epydoc annotations to public fields and
methods of Client. Also adds epydoc as a dev dependency, and adds
`make epydocs`.

It probably isn't necessary to add this documentation to our internal
methods, and I won't bother with browser for now, but I think it's
useful to have for our user-friendly interface. (Both for users directly
and through IDEs that recognize the type annotations, like PyCharm CE.)

resolves #52 - let's forget about slugs and message sets for now. We can
reply to messages and a weak set of current messages might be pointless.

ref #65, #43 - event/message iterators/contexts are still outstanding.
@Manishearth
Copy link
Owner

Hm, why is Travis timing out? /me hits retry

@Manishearth
Copy link
Owner

Still failing.

@banksJeremy
Copy link
Collaborator Author

I set a more aggressive timeout on requests than we previously had (10s), and it was causing some failures. I increased it in a later commit, and that build passed your Travis.

@Manishearth
Copy link
Owner

Ah, I see. Carry on, then :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@banksJeremy banksJeremy

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@banksJeremy @Manishearth