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

ActiveResource support #49

Open
guilhermesilveira opened this issue Feb 10, 2011 · 8 comments
Open

ActiveResource support #49

guilhermesilveira opened this issue Feb 10, 2011 · 8 comments

Comments

@guilhermesilveira
Copy link
Member

Re-write active support on top of Restfulie with restfulie capabilities

@rurounijones
Copy link

Would definitely make Restfulie easier to integrate into current projects if we could have a Restfulie'ised ActiveModel implementation.
+1

@guilhermesilveira
Copy link
Member Author

Do you think it should be the exact same API interface? Wanna help us on this one? I can show you the way
Regards

@rurounijones
Copy link

Well, unless I am mis-understanding anything, as of Rails 3 ActiveResource is also based off of ActiveModel so I think any restfulie implementation should at least be compatible with the basics of ActiveModel and pass the ActiveModel Lint test.

Unfortunately I have too many things to do at the moment so cannot commit time to this (Plus I am not a great developer so that is probably a good thing.)

I may have some time in the future (dev skill issues aside.)

@svoynow
Copy link

svoynow commented Sep 9, 2011

guilherme, I might be interested in helping out with this one. I've got a project that may find it useful. Any pointers to a good place to start digging?

@guilhermesilveira
Copy link
Member Author

Sure! Download the activeresource code. Then copy and paste the test suite.
Then you can find the place where AR invokes net... Just comment it and all
tests should fail. In that place ou can implement the code to make one
work... Then the second.

If you get the first one working let me.know already!

Guilherme Silveira
On Sep 8, 2011 10:35 PM, "svoynow-lz" <
[email protected]>
wrote:

guilherme, I might be interested in helping out with this one. I've got a
project that may find it useful. Any pointers to a good place to start
digging?

Reply to this email directly or view it on GitHub:
#49 (comment)

@svoynow
Copy link

svoynow commented Sep 13, 2011

Not sure if I followed that, can you say a little more?

@rurounijones
Copy link

One possible way of doing this is to take activeresource and replace the back-end code that uses normal http libraries with code that uses restfulie, that would be the first step. So...

  1. Clone the rails/rails repository and run the activeresource tests (to make sure everything is working)
  2. Find the code that calls Net::HTTP (Some examples in this file: https://github.com/rails/rails/blob/master/activeresource/lib/active_resource/connection.rb)
  3. Comment out all these lines of code
  4. Re-run the tests and watch them fail
  5. Replace the commented out lines with equivalents that use restfulie

That would be the first stage (IMHO) to getting a restfulie backed implementation.

@guilhermesilveira
Copy link
Member Author

Hi Jeffrey

Exactly! Thanks for clarifying

Guilherme Silveira
On Sep 12, 2011 11:23 PM, "Jeffrey Jones" <
[email protected]>
wrote:

One possible way of doing this is to take activeresource and replace the
back-end code that uses normal http libraries with code that uses restfulie,
that would be the first step. So...

  1. Clone the rails/rails repository and run the activeresource tests (to
    make sure everything is working)
  2. Find the code that calls Net::HTTP (Some examples in this file:
    https://github.com/rails/rails/blob/master/activeresource/lib/active_resource/connection.rb
    )
  3. Comment out all these lines of code
  4. Re-run the tests and watch them fail
  5. Replace the commented out lines with equivalents that use restfulie

That would be the first stage (IMHO) to getting a restfulie backed
implementation.

Reply to this email directly or view it on GitHub:
#49 (comment)

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

No branches or pull requests

3 participants