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

Delete chat room #71

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Jhanvi528
Copy link

@Jhanvi528 Jhanvi528 commented Feb 9, 2022

Have you read the Contributing Guidelines on Pull Requests?

I have gone through the Guidelines

Description

added delete functionality for each room

Checklist

  • I've read the contribution guidelines.
  • I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests

#69

@vercel
Copy link

vercel bot commented Feb 9, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/soumik2012/chathub/F4Z9f18RstPy9Cyz14VMVUkVnCJw
✅ Preview: Failed

[Deployment for 7c9339b failed]

@Jhanvi528
Copy link
Author

Jhanvi528 commented Feb 9, 2022

@heet-kumar sir I have added Homepage and delete icon functionality for each contact but even after deleting id from firebase I am not able to navigate to Homepage again as it is an async function which is causing an error whose image is also attached. Can you please guide me what should I do next? :)

ch1

ch2

ch3

ch4

@heet-kumar
Copy link

@Jhanvi528 your PR is getting conflict with Vercel Deployment please try to resolve it

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.

2 participants