-
Notifications
You must be signed in to change notification settings - Fork 183
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
No active maintainer #229
Comments
I can probably help. I don't have much time but I use the bundle a lot and I could help fix some issues. Are you searching for help or for a complete maintainer? Is there a way to talk with the ex maintainers for a better understanding of the bundle internals? |
I would volunteer as well. We use the bundle a lot in internal project. I am not too familiar with the actual internals, but can catch up as necessary. |
me too. |
Let's do something then, we are three, we can work on the current issues. @merk you seem to have worked on a 3.0-dev branch, but I can't see it anywhere. Is it done or did you give up on it? |
I can help too if this bundle is splited into component. |
I am sitting as project manager at work at the moment, so time is relatively sparse. Will read up on the internals and existing issues and see what I can start doing. |
I was working on something previously but to be honest I'm not sure where I was at or if it was worth using it to continue. |
Regarding what we're after - any help is better than none. |
If @zocimek is happy to continue and with the help offered here, we can probably remove the notice in the readme. |
I think we can continue maintain bundle. For now I'm a little bit busy but I think I can find some time to review changes and fix small bugs. With help it should be much better. |
@zocimek As you know better the bundle than us, you should probably assign us some tasks to do, as, for instance, I don't know which are still valid or not (some issues are older than a year). |
Hi, |
I agree a change is necessary but I think changing maintainer would be better than forking. The problem with forking is that we would lose the "renown" of FOS and the MessageBundle. |
+1 for changing the maintainer |
The thing is we need a current maintainer to change. Is there anyone reading? Is the bundle really dead? |
The trouble is the current FOS members dont have time to maintain this bundle (and probably arent using it). Ultimately, to become promoted to maintainer we'd want to see some activity: forking and merging outstanding PRs or making improvements would indicate this - then that code could be merged back here. My opinion is that we dont want to keep appointing maintainers who will disappear again. |
Just dropping here to mention I planned on using this bundle. So changes would be appreciated yes! Is the current version a working one ? |
@merk So should we fork it and work on it on our own then? I'm using this bundle pretty regularly and some ideas (as making a library as a base of the bundle for instance) are great ideas IMO. There are a lot of possibilities :) . I agree we don't want temoporary maintainers, but for the moment the bundle does not evolve at all, so even a bit of dynamic would be great. |
My suggestion at this point would be to fork the bundle, work on your fork for some time making changes you think would improve the bundle and propose to become the maintainer linking to your work - that means your work can be merged into a new major release of the bundle here. Alternatively, sending PRs that fix peoples issues or add new functionality and pinging me or @zocimek and in the short term we can merge them. My personal approach here would be to add some functional testing because there is very little of that in the bundle. |
I forked the bundle and I'll start to work on it there: https://github.com/tgalopin/FOSMessageBundle However, on the main repo, some issues could be closed such as:
Perhaps now is the moment to think of a new version of the bundle based on the current one concepts but using a framework independant library. I think it could be a great project but such a changement require motivated people. Are there motivated people here ready to give a hand (for the library, the bundle would be very light)? |
@tgalopin I'm interested, what kind of help do you need? |
The kind of help to rewrite the bundle as a library :) I think the steps would be:
I started a repository here to discuss the library: https://github.com/tgalopin/FOSMessage So the main and first step for the moment is to review all the features the current bundle have and what we want to add / remove to these features. I started a discussion here: https://github.com/tgalopin/FOSMessage/issues/1 |
I started to explain what I think the library should be, but I need opinions, thoughts and critics: I also need feedbacks of how you currently use the bundle: what you need, what you think that should be implemented, things like that. The new version won't be fully compatible with this version so we can invent new things and resolve new problems. Thanks by advance! |
I ported almost all the concepts of the current bundle in the library. I changed some things and add others to match with the idea of the library (it's not a bundle anymore after all :) ): https://github.com/tgalopin/FOSMessage I am still currently working a lot on the code, don't hesitate to give a hand. For the moment, the next tasks are:
What is great with this library is that it has only one light dependency (on Doctrine collections), it's very easy to reuse it without any framework. I didn't implement things like SPAM and security: I didn't implement SPAM for the moment in order to stabilize the library before, I won't implement security in the library as it's the role of the framework / developer. Please give feedbacks, it's my first shared library, I want to improve it as much as possible :) ! Thanks by advance! @redayoub : do you use Propel or Doctrine ODM? |
Can I help? |
Yes, don't hesitate! Either by contributing on the library I'm working on or by contributing to this repository (but I can't be sure your pull request will be merged any time soon). |
@tgalopin it's a good idea to write documentation for your library. |
@tgalopin great work... But we need some documentation :D |
I'm working on it, it should be available soon in an early version :) . |
I worked a lot on the library yesterday and today, it's improving a lot while far from being ready for the moment. If you want to try it and play with it, you can use the examples: https://github.com/tgalopin/FOSMessage/blob/master/Resources/doc/examples/full-example.php The concepts of the library are the same as the bundle, but the code is not exactly the same. I'll write a beginning of documentation as soon as possible (probably this night and tomorrow). I'm very interested by your opinion about the library and your ideas about it, please don't hesitate to comment, propose ideas and contribute! It's a big project and I really need inputs :) . |
Here is a beginning of documentation: https://github.com/tgalopin/FOSMessage/tree/master/Resources/doc Please try the library, I really would like to have inputs! |
@merk : do you think you could put a link to my library in the README of the bundle as the new version in development? It would improve its visibility and people would be aware of its existence. |
as per FriendsOfSymfony/friendsofsymfony.github.io#54 we have new maintainers. thanks @tgalopin and @ChristianRiesen! |
We are seeking someone who would like to assist in general maintenance of this bundle, triaging tickets and assisting in reviewing and okaying PRs.
The text was updated successfully, but these errors were encountered: