Skip to content
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

Consolidated list feature code #62

Merged
merged 9 commits into from Jan 12, 2020
Merged

Consolidated list feature code #62

merged 9 commits into from Jan 12, 2020

Conversation

@DetectiveHuman
Copy link
Contributor

DetectiveHuman commented Jan 11, 2020

Consolidated list feature handler into controller.
Fixed undefined id on edit log.

@DetectiveHuman DetectiveHuman requested a review from MattIPv4 Jan 11, 2020
src/Routes/Lists.js Outdated Show resolved Hide resolved
src/Structure/Discord/Client.js Outdated Show resolved Hide resolved
src/Routes/Lists.js Outdated Show resolved Hide resolved
DetectiveHuman and others added 4 commits Jan 12, 2020
Copy link
Member

MattIPv4 left a comment

Can we also make delete go to /lists/features/manage whilst we're here?

src/Routes/Lists.js Outdated Show resolved Hide resolved
src/Routes/Lists.js Outdated Show resolved Hide resolved
DetectiveHuman and others added 3 commits Jan 12, 2020
Co-Authored-By: Matt (IPv4) Cowley <me@mattcowley.co.uk>
Co-Authored-By: Matt (IPv4) Cowley <me@mattcowley.co.uk>
Copy link
Member

MattIPv4 left a comment

Lgtm.

@MattIPv4 MattIPv4 merged commit 0572f5e into node Jan 12, 2020
2 checks passed
2 checks passed
build
Details
WIP Ready for review
Details
@MattIPv4 MattIPv4 deleted the list-feature-controlller branch Jan 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.