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

Implements for create sub accounts with marketplace fixed #4

Closed
wants to merge 9 commits into from
Closed

Implements for create sub accounts with marketplace fixed #4

wants to merge 9 commits into from

Conversation

robertoprotasio
Copy link

No description provided.

@dynaum
Copy link

dynaum commented Mar 26, 2015

👍

@paezao
Copy link
Contributor

paezao commented Mar 27, 2015

@robertoprotasio cara você tem algum exemplo de uso do método verify no Account? Especialmente com envio de arquivos.

# @param [String] account_id the account_id of an account
# @param [String] user_token the user_token of an account

def self.search(account_id, user_token)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

O ideal seria mudar esse método pra fetch, pra seguir o padrão dos outros objetos.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dyogo que implementou esse, mas nesse caso seria somente o método search?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não é mais necessário enviar arquivos de verificação para subcontas conforme informado pelo suporte da Iugu

@paezao
Copy link
Contributor

paezao commented Mar 27, 2015

Outra coisa, creio que os métodos verify e configuration não deveriam ser métodos de classe e sim instancia, o ideal também seria configurar os tokens do account no objeto em vez de ficar enviando em todos os métodos. O que você acha @robertoprotasio ?

@robertoprotasio
Copy link
Author

Segue o exemplo implementado para verify Account @paezao

image = Base64.encode64(File.open("public/uploads/tmp/1/a1.jpg", "rb").read)
@subconta = Iugu::Account.verify(
  {
    "data" => {
      "price_range" => "Entre R$ 100,00 e R$ 500,00",
      "physical_products" => "false",
      "business_type" => "Teste",
      "person_type" => "Pessoa Jurídica",
      "automatic_transfer" => "true",
      "cnpj" => "12231312312",
      "company_name" => "BRASIL S/A",
      #"cpf" => "",
      #"name" => " ",
      "address" => "Av TESTE  123",
      "cep" => "57036-456",
      "city" => "Maceió",
      "state" => "Alagoas",
      "telephone" => "3133131313 ",
      "resp_name" => "Rob GOL ",
      "resp_cpf" => "2342343234 ",
      "bank" => "Bradesco",
      "bank_ag" => "2122",
      "account_type" => "Corrente",
      "bank_cc" => "131221"
},
    "files" => {
      "id" => "data:image/png;base64,#{image}",

     "activity" => "data:image/png;base64,#{image}"
    }
  },
"ACCOUNT_ID", "USER_TOKEN" )

@pnegri
Copy link

pnegri commented Apr 9, 2015

Meu comentário sobre o files é que o base64 deveria estar dentro do método.
Eu deveria poder fazer apenas:
files =>
id => "public/uploads/tmp/1/a1.jpg"
Poderia ser algo adicional, se ele detectasse que não era um path, dae usava do jeito que veio, senão, tentava abrir o arquivo e fazer o base64.

@robertoprotasio
Copy link
Author

Interessante essa abordagem, vamos adotar para melhorar o processo.

@dyogoveiga
Copy link

Opa bom dia, na época conversei com o Marcelo se não me engano e agente ia riscutir sobre refatorações necessárias para o pull request.

@jnettome
Copy link

Pessoal, podem me dar uma dica? Não gostaria de salvar os documentos no server, gostaria só de encodar e enviar. É possível ou estou viajando?

file_params = params.require(:data).permit(files: [:id, :cpf, :activity])
file = file_params["files"]["id"]
files["id"] = "data:image/png;base64,#{Base64.encode64(File.open(file.tempfile, "rb").read)}"

Tem me retornado o seguinte:

#<Encoding::UndefinedConversionError: "\xE2" from ASCII-8BIT to UTF-8>

Minha dúvida é se isso é algo com a conversão para JSON que é feita com os dados da request quando postados pro Iugu pela gem. Alguma idéia?

Obrigado!

@jnettome
Copy link

Pessoal, mais uma vez retornando o trabalho com Iugu aqui. (https://github.com/jnettome/iugu-ruby)

O que falta para este PR ser aceito? @paezao @pnegri

@alepaez
Copy link
Contributor

alepaez commented Oct 27, 2015

Oi @jnettome,

O @paezao colocou um comentário no arquivo lib/iugu/account.rb pedindo para trocar o nome método 'search' para 'fetch', seguindo o padrão do resto da lib.

@lucasprag
Copy link
Contributor

Oi @jnettome mil desculpas pela demora, mas você se importaria em resolver os conflitos e também mudar o nome do método search para fetch?
Assim a gente pode mergear esse PR.

Como vc pode ver estamos retomando esses projetos de integração (começando com woocommerce e iugu-ruby) e a partir de agora daremos mais atenção aos PRs.

Muito obrigado por tudo =)

@lucasprag
Copy link
Contributor

ops, agora que percebi que o meu comentário acima era para o @robertoprotasio e o @dyogo .

Vocês ainda tem interesse me mergear esse PR? Resolvendo os conflitos já conseguimos fazer o merge.

Obrigado =)

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

Successfully merging this pull request may close these issues.

None yet

9 participants