fabriciocolombo / delphi-rest-client-api

A Delphi REST client API to consume REST services written in any programming language.
Apache License 2.0
380 stars 182 forks source link

Novas Funcionalidades #8

Closed ezequieljuliano closed 9 years ago

ezequieljuliano commented 11 years ago

Boa tarde Fabrício.

Achei muito interessante sua biblioteca e estou começando a utilizá-la para testes. Fiz a adição de algumas nova funcionalidades como converter DataSet para Json e métodos POST e PUT de DataSets, caso tenha interesse pode estar adicionando ao projeto principal.

fabriciocolombo commented 11 years ago

Olá Ezequiel,

Muito interessante as funcionalidades que você implementou. Gostaria de incluir suas alterações no projeto, mais preciso que você escreva alguns testes unitários para as funcionalidades que você criou.

Poderia criar esses testes?

Abraços

2013/5/8 Ezequiel Juliano Müller notifications@github.com

Boa tarde Fabrício.

Achei muito interessante sua biblioteca e estou começando a utilizá-la para testes. Fiz a adição de algumas nova funcionalidades como converter DataSet para Json e métodos POST e PUT de DataSets, caso tenha interesse pode estar

adicionando ao projeto principal.

You can merge this Pull Request by running

git pull https://github.com/ezequieljuliano/delphi-rest-client-api master

Or view, comment on, or merge it at:

https://github.com/fabriciocolombo/delphi-rest-client-api/pull/8 Commit Summary

  • Gitignore DCU
  • Adding DataSet to convert JSON and REST methods to DataSet

File Changes

  • M .gitignorehttps://github.com/fabriciocolombo/delphi-rest-client-api/pull/8/files#diff-0(10)
  • M packages/XE3/DelphiXE3_RestApi.dpkhttps://github.com/fabriciocolombo/delphi-rest-client-api/pull/8/files#diff-1(6)
  • M packages/XE3/DelphiXE3_RestApi.dprojhttps://github.com/fabriciocolombo/delphi-rest-client-api/pull/8/files#diff-2(7)
  • A src/DataSetToJsonConverter.pashttps://github.com/fabriciocolombo/delphi-rest-client-api/pull/8/files#diff-3(134)
  • M src/HttpConnectionIndy.pashttps://github.com/fabriciocolombo/delphi-rest-client-api/pull/8/files#diff-4(1)
  • M src/RestClient.pashttps://github.com/fabriciocolombo/delphi-rest-client-api/pull/8/files#diff-5(95)
  • M src/RestUtils.pashttps://github.com/fabriciocolombo/delphi-rest-client-api/pull/8/files#diff-6(3)

Patch Links:

- https://github.com/fabriciocolombo/delphi-rest-client-api/pull/8.patch

Atenciosamente

Fabricio Colombo

Desenvolvedor Delphi e Java Maringá - PR Blog: http://fabriciodev.blogspot.com [image: Siga-me no Facebook] http://facebook.com/fabriciodev Siga-me no Facebook http://facebook.com/fabriciodev [image: Siga-me no Twitter]http://twitter.com/fabriciodev Siga-me no Twitter http://twitter.com/fabriciodev http://fabriciodev.blogspot.com/ http://fabriciodev.blogspot.com/

RobertoSchneiders commented 9 years ago

@ezequieljuliano we are very glad that you send this PR. Contributors are always welcome. Unfortunately the commit messages need to be in english, and, it would be nice if you can write some tests for it. I suppose you are not interested in contribute to this project anymore, after this long time, so, I am closing this PR. Feel free to re-open it if you want.