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

working with RC 1.3, 1.2, 1.X and 0.9 by deprecating the rcmail usage #26

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mckaygerhard
Copy link

Copy link
Owner

@sblaisot sblaisot left a comment

Choose a reason for hiding this comment

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

Thank you for your PR.

I don't understand very well why you want to replace rcmail by rcube.
I just tested the plugin with RC 1.4.2 and it works pretty well like this.
Can you point me to a deprecation or removal notice somewhere in roundcube code or mailing list ?

@@ -1,3 +1,6 @@
v0.5 :
Copy link
Owner

Choose a reason for hiding this comment

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

Please do not include version release information in your PR as you don't know how it will be named (could be 1.0 or 0.4.1). Prefere using unreleased: here, I will name it when releasing.

Copy link
Author

Choose a reason for hiding this comment

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

well there's no problem! always you can override by a new commit by yourselft.. that's all


automatic_addressbook plugin version 0.2 has been tester against roundcube 0.5 and 0.6.
above versions have been tested against roundcube 0.8.0 and 0.8.1 with larry skin and mysql backend.
automatic_addressbook plugin version 0.2 has been tester against roundcube 0.9
Copy link
Owner

Choose a reason for hiding this comment

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

tester -> tested.
Please do not update this one concerning automatic_addressbook v0.2 (an old version) or remove it completely

Copy link
Author

Choose a reason for hiding this comment

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

as i said.. can you merged and later just updated by yourselft? can you explain me more in detail?

--------------------------

you must use files before commit ca622de3364b996c655eea2e7724ce6e325e9b9d for older versions.
Copy link
Owner

Choose a reason for hiding this comment

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

Prefer reference to plugin version rather than commit id

Copy link
Author

Choose a reason for hiding this comment

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

i used commit due that commit included some changes! lasted older version still have minor issues that i'm working on

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.

RC 1.3.4 with automatic_addressbook
2 participants