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.

Sign up for GitHub

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

Jump to bottom

Migration of printer_tray to 8.0 #13

Merged
merged 21 commits into from Jan 20, 2015
Merged

Migration of printer_tray to 8.0 #13

merged 21 commits into from Jan 20, 2015

Conversation

guewen
Copy link
Member

@guewen guewen commented Nov 17, 2014

Migrated the module printer_tray.

@guewen guewen force-pushed the 8.0-printer_tray-migr branch 2 times, most recently from c7c0a2a to c9d719f Compare November 18, 2014 09:03
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling a84290b on guewen:8.0-printer_tray-migr into 8c1362e on OCA:8.0.

@guewen
Moved printer_tray from __unported__ to root folder
ab7ccba
@guewen
Set printer_tray to installable
fc884fe
@guewen
Extract the module's description in a REAME.rst file
b6fea15
@guewen
base_report_assembler seems not to be a dependency for printer_tray
e9adf1b
@guewen
print_options is now in printing, new api for ir.actions.report.xml
49a7599
@guewen
new api for res.users
0a5971e
@guewen
new api for printing.tray
6d3c5c8
@guewen
new api for 'printing.report.xml.action'
3ef4432
@guewen
Started to migrate printing.printer to the new API
aef795d
@guewen
Update the trays from cups, use the new extracted methods in
57b9130 
base_report_to_printer
@guewen
unused import
2da9855
@guewen
add trays in printers view
fb01704
@guewen
Add the printer tray choice in the 'My preferences' popup
b707791
@guewen
Updated translations
9b763de
@guewen
Wrong column name
ca85601
@guewen
Missing import
861d8d4
@guewen
Remove the PPD file once read, otherwise they accumulate
d3b0c93
@guewen
Security file was not loaded
a4d58ea
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling a4d58ea on guewen:8.0-printer_tray-migr into 31c61a8 on OCA:8.0.

@yvaucher yvaucher added the 8.0 label Dec 12, 2014
@yvaucher
Copy link
Member

👍

@guewen
Add the trays in printing.report.xml.action view
80b734b 
So they can be configured per user and per report
@coveralls
Copy link

Coverage Status

Coverage decreased (-1.63%) when pulling 80b734b on guewen:8.0-printer_tray-migr into 31c61a8 on OCA:8.0.

pedrobaeza
pedrobaeza reviewed Jan 20, 2015
View reviewed changes
printer_tray/README.rst
config and on specific config per user per report.

This allows you to dedicate a specific paper source for exemple for
preprinted paper such as payment slip.
Copy link
Member

Choose a reason for hiding this comment

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

You should consider to add OCA description template

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.39%) when pulling cfdfc8e on guewen:8.0-printer_tray-migr into 31c61a8 on OCA:8.0.

@pedrobaeza
Copy link
Member

Thanks 👍

When Travis goes green, I'll merge.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.45%) when pulling cacc542 on guewen:8.0-printer_tray-migr into 31c61a8 on OCA:8.0.

pedrobaeza added a commit that referenced this pull request Jan 20, 2015
@pedrobaeza
Merge pull request #13 from guewen/8.0-printer_tray-migr
2f71ee7 
Migration of printer_tray to 8.0
@pedrobaeza pedrobaeza merged commit 2f71ee7 into OCA:8.0 Jan 20, 2015
@guewen
Copy link
Member Author

guewen commented Jan 20, 2015

Thanks @pedrobaeza

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
8.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@guewen @coveralls @yvaucher @pedrobaeza