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

13.0 mig helpdesk mgmt #104

Closed
wants to merge 91 commits into from

Conversation

emagdalenaC2i
Copy link
Contributor

No description provided.

aitorbouzas and others added 30 commits October 17, 2019 15:51
* Add ticket model and its fields
* Adding readme with different tasks
[FIX] description field
* models category and team related with users
* many2many field team to categories
* description in team and category
* [ADD] Readme

* Generate readme with OCA Maintainer Tools
[FIX] 'heldesk' in __manifest__.py

* [ADD] Stage model and data (OCA#16)

* [FIX] Flake8 and naming conventions
[FIX] Remove ACSONE
[FIX] Build error with portal.mixing. This will be added in a second module

* [FIX] Fix travis errors
* Category in manifest
* Added group category in data
* Added groups
* Added rules
* Added model access
[ADD] Create structure menu
- Active field
- Automatic number with sequence
* Fix menu security
* Channel data

  * Add 4 channels (web, email. phone and other)
  * Add some help for channel in ticket field

* Active fields in required models
* Fold value in stage data
* Demo data
* Add views
* [ADD] functionality of datetime fields
* [ADD] test for datetime fields [FIX] error in demo data
* [FIX] flake8
* Fix tickets and naming conventions (OCA#46)
* Save sequence number in in 'number' field instead of in helpdesk_sequence.
* Also fixed tests and added a small test to check if a number is added in a ticket when we create one.
* Add many views and fix others (OCA#49)
* [ADD] Helpdesk: Añadida plantilla y función de envío de mail.

* [FIX] Fix the sending user email
* Add date to list view of tickets (OCA#50)
* Tag views (OCA#51)
* Add mail template automatic sending when changing stages (OCA#52)
* [FIX] Fix visibility on stage field
* [FIX] Add admin to helpdesk manager group by default
* new: added a smartbutton to the res_partner form view in order for a given partner to access its tickets

* fix: migration to py3.5 - removal of f-strings

* chg: partner smartbutton - linked with active records by default !minor

* [ADD] Dashboard view

* travis fix

* fix: beautified the example data xml !minor

* fix: flake8'd the helpdesk_ticket_team file to be standaridized as per oca guidelines !minor

* fix: removed duplicated helpdesk_ticket_action record id !minor

* fix: beautified the helpdesk.ticket.tag views !minor
* [FIX] Minor changes.

* fix minor changes

* fix 'cancelled' string

* fix team_id and user_id behavior

* fix travis
* Development status
* Put some XML-ID references in XML files not fully qualified for avoiding problems.
* Change XML-ID reference new module in code.
* Invalid fields in demo data
* README
* Contributor
@pedrobaeza pedrobaeza added this to the 13.0 milestone Oct 17, 2019
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 17, 2019
2 tasks
@@ -139,7 +139,7 @@ def copy(self, default=None):
return res

def write(self, vals):
for ticket in self:
for _ticket in self:
Copy link
Member

Choose a reason for hiding this comment

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

Why use a recordset loop if we are working only with vals parameter?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I do not know. It doesn't make sense, but it's like that for Odoo 11 and 12

@cubells
Copy link
Sponsor Member

cubells commented Mar 23, 2020

@emagdalenaC2i @DarioLodeiros
Are you going to continue with this or do you want I finish this migration?

<field name="name">Other</field>
</record>
<!--default alias for leads-->
<record id="mail_alias_lead_info" model="mail.alias">
Copy link
Member

Choose a reason for hiding this comment

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

You can not use this id "mail{alias_lead_info" because it is used on crm module, and this do that module crm will be uninstallable.

@cubells
Copy link
Sponsor Member

cubells commented Apr 1, 2020

Supersede in #117

@cubells
Copy link
Sponsor Member

cubells commented Apr 1, 2020

@angelmoya
can you test this new pr (#117)?

@emagdalenaC2i
Copy link
Contributor Author

@cubells I close this PR and will try to test #117 soon

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