[11.0] [MIG] Migrate donation #46
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ed wizard Generate Mass Journal in mass module 3. Fixed a bug in Stay module room name was empty in tree views
…n number may be generated in a sequence per user. By default, number is generated in the donation sequence
… for foreign currencies in donations. Add mass validation wizard (validate mass lines and create account moves) Add conventual information on mass lines. Small usability improvements.
…ass line if certains scenarios Fix a bug when re-validating a donation.
… validation Add "Back to draft" on validated mass transfers
…otal_company_currency on donation.donation Remove view_type from action definition
…field 'Amount in Company Currency' on donation.line Add reporting menu for donations
…oices, cf account module)
…d French translation by Brother Irénée Move donation_data.xml to donation_demo.xml
…plate/product.product in XML.
…th the rest of Odoo v8 rename variable filter_to_donation filter_donation Add filter for Tax receipt Inherit search view of product.template instead of product.product
…Update strings and translation.
… Add Contact menu entry in Stay menu PEP8 stuff
Add tree view on donation.report for drill-through Fix @api.model -> @api.multi
DB with demo data can be populated with current state, as there's no chart template, and donation demo data search for a journal, which is not present if no localization module is installed. This trick assigns before that demo data a country to the main company and execute again the autoinstall function through a helper method.
Remove @api.multi when not needed PEP8 fixes Replace openerp by odoo
Remove dead comment
donation_bank_statement: add domain type=bank on donation_credit_transfer_journal_id donation: Small code improvement on income account selection from product PEP8 fix
|
FYI Added OCA dependencies File, Because Travis Failing some dependencies issues.
|
nikul-serpentcs
force-pushed
the
11-donation
branch
from
4b02220
to
65e252a
Compare
nikul-serpentcs
force-pushed
the
11-donation
branch
from
dd5f5e6
to
ba586e7
Compare
|
@JayVora-SerpentCS @pedrobaeza @tafaRU Could you please review it? |
alexis-via
approved these changes
Oct 28, 2018
|
I made a rapid review, that's fine for me. There has been several improvements in the 10.0 branch in the meantime, which would be great to up-port, but this can be done in a later PR, I don't want to post-pone this for an up-port. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Donation
This module handles donations, including in-kind donations. When you validate a donation, it will create the corresponding accounting entries.
@JayVora-SerpentCS Could you please review it?