Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The Engineering group that was set up at install is set to "Joining requires admin approval "
As a regular user I joined the engineering group. Since the only indication of my group status says <a>Leave group</a> I would expect to be a member and wonder why I can't do anything.
Shouldn't there be saying "membership pending approval by group organizers" or the like?
Comment | File | Size | Author |
---|---|---|---|
#2 | 2020247-2-pending-approval-message.patch | 1.51 KB | ezra-g |
Comments
Comment #1
ezra-g CreditAttribution: ezra-g commentedExcellent point! Thanks for the suggestion.
Moving to the "Groups" component.
The referenced text is actually new as of 3.3 with the intention of making this more clear. Marking for triage in the 3.3 radar.
Comment #2
ezra-g CreditAttribution: ezra-g commentedThe attached patch changes the pending membership to displaying the text "Your membership request is pending review by a group organizer" as a non-link.
Comment #3
behoppe333 CreditAttribution: behoppe333 commentedVery happy to see this issue on the table.
On a related note, for groups that hide contributed content from non-members, the commons browsing widget says things like "There are no posts to display" when it would be more helpful to say "You do not have access to view this content." Should I create a separate issue for that?
Comment #4
ezra-g CreditAttribution: ezra-g commented@behoppe333 Thanks. Are you able to review the patch?
Yes, please create a new issue for your suggestion. Thanks!
Comment #5
behoppe333 CreditAttribution: behoppe333 commented@ezra-g, yes -- with some coaching I will be able to review the patch. Last week I managed to apply my first patch with help from devincarlson :-)
The coaching is about how best to arrange my instance (or instances?) of Drupal Commons so that I can both (1) create a new site from scratch intended for production launch soon after release of C3.3 and (2) test the latest dev code so that C3.3 is as awesome as possible. I would prefer to manage a single instance of Drupal Commons, but I am struggling. It seems like patches require me to have local git repos for each relevant project (e.g., Commons Documents, Commons Groups, etc) and those (it seems) sit squarely inside the destination folders where my "drush make" rebuilds go. I am also trying to establish a baseline site DB (i.e., my soon-to-be-production C3.3 site) and I am not sure if it's reasonable to do that and each day (or week) update the code that sits on top of that DB.
(At risk of TMI, here are reasons why I am trying to do all those at once: I am simultaneously (a) creating a brand new community site for my company, (b) planning to use Commons 3.3 for that community site, (c) using Commons 7.x-3.x-dev to create mock-ups for key stakeholders of the new site, (d) establishing baseline Drupal settings/DB for the new site, (e) testing Commons 7.x-3.x-dev, and (f) learning Drupal, drush, and git as fast as I can.)
Comment #6
behoppe333 CreditAttribution: behoppe333 commentedI have done very simple test of patch in #2 and it looks good, btw. :-)
Comment #7
ezra-g CreditAttribution: ezra-g commented#2 is committed: http://drupalcode.org/project/commons_groups.git/commit/4a64bb1.
@behoppe333, We'd be happy to discuss your development workflow in a new issue. Thanks!
Comment #8
behoppe333 CreditAttribution: behoppe333 commentedI created a fresh instance of Commons from scratch on July 9 and this bug has resurfaced. I verified that my instance has the commit from #7. I could be wrong but my hunch is funky interaction happening between recent commits:
Comment #9
japerryI too can reproduce this using the steps in https://drupal.org/node/2019057#comment-7561585 (#17). It just says 'join group' even though the code is in there.
Updating groups git to this:
http://drupalcode.org/project/commons_groups.git/commit/771a331e25cbec3d...
Causes this error to come back.
Comment #10
japerryFixed the regression caused in the previous commit. this should be fixed now!
http://drupalcode.org/project/commons_groups.git/commit/9b2e4fe
Comment #11
RobKoberg CreditAttribution: RobKoberg commentedworks for me now as of this mornings build.
Comment #12
behoppe333 CreditAttribution: behoppe333 commentedworks for me too. thanks :-)