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

User Favourites: Remove deleted applications #3697

Closed
1 of 5 tasks
troytop opened this issue Jul 8, 2019 · 2 comments · Fixed by #4431
Closed
1 of 5 tasks

User Favourites: Remove deleted applications #3697

troytop opened this issue Jul 8, 2019 · 2 comments · Fixed by #4431
Labels

Comments

@troytop
Copy link

troytop commented Jul 8, 2019

Frontend Deployment type

  • Cloud Foundry Application (cf push)
  • Kubernetes, using a helm chart
  • Docker, single container deploying all components
  • npm run start
  • Other (please specify below)

Expected behaviour

When I delete an application from cf, I expect the box for this application to dissapear from the Favourites list in the Home view.

Actual behaviour

The application disappears from the Application view, but persists on the Home view. When I redeploy the application and click the Favourite star on the newly (re)deployed application, I get a duplicate entry on the Home view. One of them leads to the new application, the other leads to main Applications view.

Context

Possible Implementation

@KlapTrap
Copy link
Contributor

Good spot.

Implementation notes:

Any entity that the user deletes during their session will be marked as deleted in the entity request section of the store. The user favourites sync should notice the deleted flag and remove any orphaned favourites.

If an entity that has been favourited is deleted outside of the session i.e. by someone else, and if the user clicks that favourite then we should show some 404 page or at least some messaging as to why they've been taken to the application wall.

@KlapTrap
Copy link
Contributor

See #3699 for 404 page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants