Skip to content

Conversation

@ElishebaW
Copy link
Contributor

@seanmarcia I was hoping I could get some help on this.

Issue #1
Here's what I'm trying to do:
I want to add a toggle feature using javascript to add the that would make the div with the checkboxes and remove button visible with CSS when you click the remove member link and remove the classes when you click the link again to make the div hidden. Right now, the CSS class isn't being added to the class list when I click the button nor is the class being removed. There are no errors in the console.

I tried this https://alligator.io/js/classlist/ , with the toggle and it didn't work. So I went back to the class List Add and Remove.

I'm also open to other solutions.

Issue #2

screen shot 2018-01-13 at 4 04 26 pm

I can't for the life of my figure out why I'm getting this. I did have a remove_member method and route, but I have since removed them. Any ideas?

@seanmarcia
Copy link
Member

Hey @ElishebaW I'll take a look shortly. I'm trying to remember the full context to this.

On first pass at looking at your code I think that this part: @organization.users >> user doesn't make sense for removing a user.

@seanmarcia
Copy link
Member

Hey @ElishebaW can you push up a working version without the javascript toggle? I think it would be an easier starting place.

@ElishebaW ElishebaW force-pushed the Org-owner-can-remove-members-from-an-organization branch from b105ed2 to 33d26e0 Compare January 20, 2018 01:10
@ElishebaW
Copy link
Contributor Author

ElishebaW commented Jan 20, 2018

@seanmarcia Do you mean like my last commit? For the working copy? The error only appeared when I clicked the Remove button. Maybe we can pair sometime?

@seanmarcia
Copy link
Member

Let's do it! I'll ping you in slack :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants