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

Remove list of deprecated modules, update readme. #225

Closed
wants to merge 1 commit into from
Closed

Remove list of deprecated modules, update readme. #225

wants to merge 1 commit into from

Conversation

feketemihai
Copy link
Member

No description provided.

@coveralls
Copy link

coveralls commented May 12, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 87.912% when pulling 1efae67 on feketemihai:deprecate into 3f72a72 on OCA:8.0.

@feketemihai
Copy link
Member Author

feketemihai commented May 12, 2016 •
edited
Loading

Follow up of #224

@feketemihai feketemihai added this to the 8.0 milestone May 12, 2016
@ghost
Copy link

ghost commented May 12, 2016

👍

@nhomar
Copy link
Member

nhomar commented May 12, 2016

Hello.

IS this possible you can in the commit message explain why the modules is deprecated?

Deprecate a module without a single explanation (even if it is obvious) is a bad practice.

After a proper explanation of the deprecation +1 for me.

@dreispt
Copy link
Member

dreispt commented May 12, 2016

I think we should have a separate PR for each module removed.
That allows to leave a better trail of why it was removed, and allows people interested in each particular module to present arguments on the corresponding discussion.

@feketemihai
Copy link
Member Author

I will create separate PR with my opinion why it should be deprecate, i will create one PR for aeroo report removed in #63 from 7.0

sambarros pushed a commit to sambarros/hr that referenced this pull request Jul 26, 2018
@guewen
Merge pull request OCA#225 from TelmoSenseFly/BSSFL-330
b77abef 
[BSSFL-330] Payment mode invoice bank details
Mraimou pushed a commit to camptocamp/hr that referenced this pull request Nov 25, 2019
@nbessi
Merge pull request OCA#225 from gcoudu/bso_backbone_bundle_dealsheet
756364a 
BSO Backbone write optimized
Mraimou pushed a commit to camptocamp/hr that referenced this pull request Nov 25, 2019
@nbessi
Release 10.13.0
0509959 
**Features and Improvements**
  * Improvements in Mailchimps webhook

**Bugfixes**
 * Fix in Dashboard, Backbone, Mailchimp
     PR OCA#225 OCA#226 OCA#228 OCA#22
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
needs review
Projects
None yet
Milestone
8.0
Development

Successfully merging this pull request may close these issues.
4 participants
@feketemihai @coveralls @nhomar @dreispt