Skip to content
This repository was archived by the owner on Feb 10, 2022. It is now read-only.

Country support and improvements#11

Open
Luciano Becerra (lubc) wants to merge 7 commits into
mercadolibre:masterfrom
lubc:country_support_and_improvements
Open

Country support and improvements#11
Luciano Becerra (lubc) wants to merge 7 commits into
mercadolibre:masterfrom
lubc:country_support_and_improvements

Conversation

@lubc

Copy link
Copy Markdown

Hey,

This PR fixes the issue #10 and also adds some improvements.

All the changes:

  • Allows to set a country code in auth_url method
  • Removes config.yml file and declares all the constants in lib/constants.rb
  • Adds Rspec dependency, updates syntax in spec/meli_spec.rb and adds coverage for auth_url
  • Fixes indentation
  • Updates README.md

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

That's necesary

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

These changes look good! 👍

@lubc

Copy link
Copy Markdown
Author

Raphael Maurício Cerqueira (@Phaael) Rafael Prenzier (@prenzier) Could you review this? Thanks!

@r1goo

Copy link
Copy Markdown

Thanks bro!

@r1goo

Copy link
Copy Markdown

Why are taken so long to merge this thing up? I just cant solve the problem with the BR redirection.

@raphaeljlps

Copy link
Copy Markdown

This is so frustrating, I'm trying to build something with mercadolibre
and lost a ton of time having issues that this PR fixes,
is this repo not maintained anymore?

could someone from ML please, merge this wonderful PR?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants