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

Chatroom rename feature #529

Merged
merged 18 commits into from
Oct 9, 2023
Merged

Conversation

staticGuru
Copy link
Contributor

@staticGuru staticGuru commented Oct 9, 2023

  • What kind of change does this PR introduce?
    Feature- Closes- 🚀 Feature: Chat room rename functionality #495

  • Why was this change needed?
    - The chat room renaming feature significantly enhances the user experience by providing a valuable means of contextualizing conversations within their respective chat rooms.

  • Other information:
    Demo:

Screen.Recording.2023-10-09.at.4.00.42.PM.1.mov

@vercel
Copy link

vercel bot commented Oct 9, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 9, 2023 10:41pm
nextra-docsgpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 9, 2023 10:41pm

@vercel
Copy link

vercel bot commented Oct 9, 2023

@staticGuru is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

Merging #529 (a998db0) into main (e108833) will increase coverage by 0.02%.
Report is 29 commits behind head on main.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##             main     #529      +/-   ##
==========================================
+ Coverage   17.50%   17.53%   +0.02%     
==========================================
  Files          59       59              
  Lines        2719     2726       +7     
==========================================
+ Hits          476      478       +2     
- Misses       2243     2248       +5     
Files Coverage Δ
application/api/user/routes.py 26.81% <28.57%> (+0.09%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dartpain
Copy link
Contributor

dartpain commented Oct 9, 2023

Great job, Thank you!

@dartpain dartpain merged commit 64a8857 into arc53:main Oct 9, 2023
10 checks passed
@staticGuru
Copy link
Contributor Author

staticGuru commented Oct 10, 2023

Hi @dartpain @tenokami
Thank you for accepting my pull requests and I'm happy to hear that you'd like to add the hacktoberfest-accepted label.

@dartpain
Copy link
Contributor

@staticGuru Will count your contributions for https://hacktoberfest.com/ even without the label. All that matters is that your contribute to repositories that participate. But I will add it for you

@staticGuru
Copy link
Contributor Author

@staticGuru Will count your contributions for https://hacktoberfest.com/ even without the label. All that matters is that your contribute to repositories that participate. But I will add it for you

@dartpain Thank you so much 💓

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

Successfully merging this pull request may close these issues.

2 participants