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

My Feeds page: connect feed deletion button to backend functionality #946

Closed
Silvyre opened this issue Apr 6, 2020 · 0 comments · Fixed by #933
Closed

My Feeds page: connect feed deletion button to backend functionality #946

Silvyre opened this issue Apr 6, 2020 · 0 comments · Fixed by #933
Assignees

Comments

@Silvyre
Copy link
Contributor

Silvyre commented Apr 6, 2020

What would you like to be added:

The implementation of the feed deletion buttons on the My Feeds page (introduced by #933) must be completed, i.e. by invoking the to-be backend logic for feed deletion that is being developed by @c3ho and @Grommers00.

Why would you like this to be added:

This is one step towards ensuring that the My Feeds page is capable of facilitating a vital administration functionality.

@Silvyre Silvyre added type: enhancement New feature or request area: front-end labels Apr 6, 2020
@Silvyre Silvyre self-assigned this Apr 8, 2020
Silvyre added a commit that referenced this issue Apr 9, 2020
* add form logic, help modal to My Feeds page

* fetch user feeds into separate form controls

* add delete feed buttons, refactor logic

* address review comments

* remove call to refresh window upon feed deletion

* refactor a useState call for consistency

* redirect non-logged-in users to 404 page without rendering My Feeds page

* add feed deletion logic to DeleteFeedDialogButton

* implement getUserFeeds() stop-gap

* fix DeleteFeedDialogButton props

* correct props destructuring in 3435b76

* remove commented-out proptypes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant