rubycorns / rorganize.it

A web app to coordinate railsgirls project groups
https://rorganize.it
GNU Affero General Public License v3.0
41 stars 23 forks source link

Group invites, closes #449 #648

Closed lislis closed 6 years ago

lislis commented 6 years ago

Is this solving an issue already in our backlog? If so, please mention which one

449

Special things about this PR to be considered

This is one of the biggest changes yet and it will take me some time to work through this.

Do you have any open Questions you need to get answered?

If you find any edge cases that should be tested, please tell me!

Please outline the areas that should be tested, if there are any:

I'll introduce the concept of an invite. Invites are temporary relationships between a person and a group, similar to memberships but, well, temporary. A user request membership to a group, an invite is created. A group member either accepts or rejects it. Both actions destory the invite with the only difference being that accept also creates a membership.

So far the plan. I had some other ideas, that's why the model is a bit weird, not sure if I keep it or throw it out.

lislis commented 6 years ago

Ok, so here is a rundown of the current workflow with screenshots

lislis commented 6 years ago

Updates! :tada: I added feature tests, but the invites controller #destory spec is not working the way I want it to. Can someone have a :eyes: ? Other than that I think this is ready to be reviewed! Comments welcome!

Here are the updated screens

Request membership

screenshot from 2018-10-09 20-23-44

Membership requested

screenshot from 2018-10-09 20-23-48

pending memberships on profile page

screenshot from 2018-10-09 20-23-57

New membership request when group admin

screenshot from 2018-10-09 20-25-41

Manage requests when group admin

screenshot from 2018-10-09 20-28-03

Emails are the same

lislis commented 6 years ago

Ok, so now it's the fun time where tests pass locally but weird errors appear on Travis

lislis commented 6 years ago

I don't know why the errors are now appearing and also in tests I haven't touched. Could it be that Travis is installing another version of some dependency or something strange like that? Anybody up for debugging?

lislis commented 6 years ago

So, I cleared all the caches on Travis and locally and still the same: local passing, Travis failing. I'm out of ideas how I could reproduce the errors. @til @PragTob any ideas?

lislis commented 6 years ago

Yay, @vsmart figured out what caused the Travis error! Apparently I deleted a part of the vcr_cassette for groups which then caused the geocoder to fail on all tests that involved creating groups. Resorting that cassette from master fixed the issue! :tada:

sareg0 commented 6 years ago