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

Added more improvements #8

Merged
merged 12 commits into from Jul 31, 2015
Merged

Added more improvements #8

merged 12 commits into from Jul 31, 2015

Conversation

macagua
Copy link
Contributor

@macagua macagua commented Jun 8, 2015

  • Updated package documentarion
  • Added i18n support
  • Added Spanish translation

@macagua
Copy link
Contributor Author

macagua commented Jun 8, 2015

@hvelarde please check out this pull request and merge if all is 👍 (ok)

cc @ericof

@macagua
Copy link
Contributor Author

macagua commented Jul 13, 2015

@hvelarde What is wrong with the plone 5 travis-ci task?, It can not pass the build task :(

I would like that it be possible to merge this improvements :) 👍

@@ -4,4 +4,19 @@ Changelog
1.0a1 (unreleased)
------------------

- Fixed errors for code analysis test of this package
Copy link
Member

Choose a reason for hiding this comment

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

@macagua please do not clutter the changelog; add only stuff useful to integrators and developers; IMO this is not useful

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok done!

@hvelarde
Copy link
Member

@macagua tests must pass first and you have to contact @jeanferri for a merge of this

@macagua
Copy link
Contributor Author

macagua commented Jul 31, 2015

@jeanferri please can check this build on travis https://travis-ci.org/plonegovbr/collective.opendata/jobs/70648696

It build job said "No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself. The build has been terminated"

jeanferri added a commit that referenced this pull request Jul 31, 2015
Added more improvements merging @macagua pull request.
@jeanferri jeanferri merged commit d540959 into plonegovbr:master Jul 31, 2015
@macagua
Copy link
Contributor Author

macagua commented Aug 1, 2015

@jeanferri thanks very you!!!

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

3 participants