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

ui fix: align budget fields of LR line form #48

Merged
merged 3 commits into from
Nov 19, 2014

Conversation

lepistone
Copy link
Member

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling f6ef7b7 on lepistone:fix-align-budget into d1361fe on OCA:8.0.

@lepistone
Copy link
Member Author

On the red travis:

It is on odoo only, on tests with assertraises. It is not related to this PR. I think I know what that is, I will address it separately. Please consider that build as green.

@lepistone
Copy link
Member Author

The red build in with odoo is due to odoo/odoo#3697. Still unrelated to this PR.

@yvaucher
Copy link
Member

@lepistone not sure about this one. Doesn't it throw description below ?

@lepistone
Copy link
Member Author

you're right @yvaucher. I'm fixing.

@lepistone
Copy link
Member Author

@yvaucher should be OK now. I also switched the budget column to the right.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e7eecb2 on lepistone:fix-align-budget into d1361fe on OCA:8.0.

@lepistone
Copy link
Member Author

@yvaucher I applied the same change in the lrl form inside the lr form

@yvaucher
Copy link
Member

👍

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling c698cbd on lepistone:fix-align-budget into d1361fe on OCA:8.0.

@nbessi
Copy link

nbessi commented Nov 19, 2014

👍

1 similar comment
@guewen
Copy link
Member

guewen commented Nov 19, 2014

👍

guewen added a commit that referenced this pull request Nov 19, 2014
ui fix: align budget fields of LR line form
@guewen guewen merged commit 2807ad1 into OCA:8.0 Nov 19, 2014
@lepistone lepistone deleted the fix-align-budget branch November 19, 2014 12:46
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

5 participants