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

Bug Report: Advertisement Deletion Not Reflected on Frontend but getting updated on Backend. #1109

Closed
aialok opened this issue Dec 1, 2023 · 4 comments
Assignees
Labels
bug Something isn't working unapproved

Comments

@aialok
Copy link
Contributor

aialok commented Dec 1, 2023

Describe the bug

  • Bug: Frontend does not reflect successful advertisement deletion despite backend update.

To Reproduce
Steps to reproduce the behavior:

  1. Goto Talawa Admin Dashboard
  2. In advertisement section
  3. Select Active or Completed Compaigns
  4. Click on Delete Button

Expected behavior

  • Upon deleting the advertisement, it is expected to be removed from the backend and no longer displayed on the frontend. However, despite deletion on the backend, the advertisement continues to appear on the frontend.

Actual behavior

  • Despite attempting to delete the advertisement on the frontend, it persists and remains visible.

Screenshots

  • Before
Delete_Bug.webm
  • After
Fixes_Delete_bug.webm

Additional details

  • There are some more issues in the advertisement section. I'm not sure why we use window.location.reload() when creating an advertisement.

Potential internship candidates
Please read this if you are planning to apply for a Palisadoes Foundation internship PalisadoesFoundation/talawa#359

@aialok aialok added the bug Something isn't working label Dec 1, 2023
@aialok
Copy link
Contributor Author

aialok commented Dec 1, 2023

@palisadoes I would love to work on this issue.

@beingnoble03
Copy link
Member

@aialok We need to remove that location.reload function. Please create issues, as required to do so.

@aialok
Copy link
Contributor Author

aialok commented Dec 2, 2023

Sure, also there are some minor bugs. I will create an issue for the same.

@aialok
Copy link
Contributor Author

aialok commented Dec 5, 2023

@noman2002 can we close this now, as the pull request has already been merged?"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working unapproved
Projects
None yet
Development

No branches or pull requests

3 participants