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

[FIX] runbot_travis2docker: Remove db parameter in URLs when t2d is used #157

Merged
merged 1 commit into from May 31, 2018

Conversation

luisg123v
Copy link
Contributor

When builds are generated using travis2docker, the database name will
always be set to openerp_test. Therefore, the db name given by default
by Runbot should not be used in the build' sign in URL.

This change ensures URLs include the db name parameter only when the
build was not generated using travis2docker.

In addition, this hides the "Connect base" button when the build was not
generated using travis2docker, because the database *-base is not
created in such cases.

…sed (Vauxoo#168)

When builds are generated using travis2docker, the database name will
always be set to `openerp_test`. Therefore, the db name given by default
by Runbot should not be used in the build' sign in URL.

This change ensures URLs include the db name parameter only when the
build was not generated using travis2docker.

In addition, this hides the "Connect base" button when the build was not
generated using travis2docker, because the database *-base is not
created in such cases.
@luisg123v
Copy link
Contributor Author

CC @moylop260

@hbrunn hbrunn merged commit 97cf7b3 into OCA:11.0 May 31, 2018
@moylop260 moylop260 deleted the 11.0-oca_db_in_url-luisg branch June 28, 2018 18:33
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