Needs review
Project:
Drupal 8 Contrib Porting Tracker
Version:
8.x-0.x-dev
Component:
Module
Priority:
Normal
Category:
Plan
Assigned:
Unassigned
Reporter:
Created:
5 Apr 2016 at 05:58 UTC
Updated:
11 Feb 2023 at 12:12 UTC
Jump to comment: Most recent
Comments
Comment #2
mohit_aghera commentedComment #3
tim bozeman commentedHiya @mohit_aghera. I'm going to close this issue. I started an 8.x branch at https://www.drupal.org/node/20967/git-instructions/8.x-1.x. Lets work on it there!
Comment #4
mohit_aghera commented@Tim Bozeman do we have list of pending issues or issues to be worked upon ?
So that we can work on it and get it ready for beta release.
Comment #5
tim bozeman commentedI just made the sending methods pluggable and added send_by_email so it's functional. I'm self QAing and hunting for issues atm.
Comment #6
webchickJust a dev release on Drupal.org, so bumping down.
Comment #7
mohit_aghera commented@Tim Bozeman
Is there anything that i can help you with to make it to beta release ?
Comment #8
tim bozeman commented@mohit_aghera Yes please! I'm not sure there's been any other eyes on this code yet.
Comment #9
mohit_aghera commented@Tim Bozeman I'll go through code and will add suggestions
Comment #10
mohit_aghera commented@Tim Bozeman
After doing quick inspection i found few things.
Let me know your thoughts on this. I can work on this.
Comment #11
tim bozeman commented@mohit_aghera Ooo ya all of those! Maybe for the email address we can prepopulate it with what's in the invite and disable the field so it's not editable.
Comment #12
mohit_aghera commented@Tim Bozeman
First patch here to disable email field during registration #2752037: Disable email address field when users joins by invitaion
Comment #13
mohit_aghera commented@Tim Bozeman
I have implemented invite resend functionality in this issue #2752085: Invite Resend Implementation
I have few more suggestions for invite entity.
Comment #14
mohit_aghera commentedAdded this issue #2752721: Invite withdraw invite Implementation for withdraw invite implementation.
Comment #15
Anonymous (not verified) commentedHi,
Is this module ready to be used in Drupal 8 ?
What is missing to have a beta version ?
Regards,
Comment #16
mohit_aghera commented@pachakamac
Module is pretty much ready, still @Tim Bozeman can confirm once.
I have also submitted few more patches for adding extra functionality.
Comment #17
tim bozeman commentedThe project I was on abandoned this functionality so I haven't had a chance to really test it out :/
Comment #18
Anonymous (not verified) commentedHi @TimBozeman, Hi @mohit_aghera,
Thank you for your answers.
I would be ready to test the module.
I want to use it for a project where I can spend times to make some experimentations.
If it can help, I could report bugs to make it stable.
Let me know if you are interested.
Regards,
Comment #19
mohit_aghera commented@pachakamac
That would be great !!
I am happy to help wherever needed.
Comment #20
mmjvb commentedComment #21
renatog commentedComment #22
renatog commentedThe Invite project was been ported to D8 and really works good.
Thank you all for contributions.
Comment #23
renatog commentedComment #24
mmjvb commentedAs long as there is only a beta available, Status should be Needs review.
As soon as a Stable release is made, it can be set to Fixed. Please see the D8CPT description for how Status works in this project. It doesn't work like the queues for Modules !
Comment #25
avpaderno