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

packagist #2

Open
Archelyst opened this issue Mar 3, 2014 · 7 comments
Open

packagist #2

Archelyst opened this issue Mar 3, 2014 · 7 comments
Labels

Comments

@Archelyst
Copy link

Hi Nicolas,

are you still maintaining this? Interested in making a package on packagist.org? I also have some changes that should be integrated. Otherwise I'd just fork and publish it myself, although I'm not sure if I'd maintain it over a longer period.

Tim

@beraneck
Copy link
Owner

beraneck commented Mar 8, 2014

Hello !

First of all, I'm still using this so yes I'm trying to maintain it. My first objective was to be able to integrate it to a project that uses composer. I thought about adding it to packagist.org but first tried to do a pull request in jamesiarmes repository (jamesiarmes/php-ews : PSR-0 version jamesiarmes#115). This PR has one year but I don't know if it will be merged one day.

To be honest I don't know what to do at this time. I've seen also a lot of forks of this repository which adds interesting improvements and corrections. If you are motivated we can bring them back to this repo and try to publish something on packagist !

Because documentation should be upgraded (I have seen something already done in another fork) and I've missed some classes during the update (also seen corrections somewhere).

If you already did some update, you can open a PR and I'll review it and integrate it to this repo as a beginning.

BR
Nicolas

@Archelyst
Copy link
Author

Good to hear this is still alive. Let's tackle this and publish a solid version on packagist. Same reason for me by the way: want to integrate via composer. I'll fork and make some changes which you can then merge. Which improvements are you referring to?

@Archelyst
Copy link
Author

Have you managed to use NTLM for Exchange 2010 or is it really not supported? We use basic auth instead here for that version. Depending on whether NTLM is possible in general I'd add either an internal switch or allow to configure the authentication method. What do you think?

@Archelyst
Copy link
Author

I merged the changes jamesiarmes made in the mean time. A lot of new types, better documentation and alphabetical sorting of attributes. Very nice.

@Archelyst
Copy link
Author

more improvements merged (by slackero, actually just yesterday :) )

@Archelyst
Copy link
Author

ok, some more improvements, willl stop for now, it's cherry picking from here on

@beraneck
Copy link
Owner

Hi Tim !

Great news ! Don't forget to make a pull request when you feel it is ready !

I'll have a look at the forks I spoke about in my first message.
I'll also update the documentation (at least the readme).

BR
Nicolas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants