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

feat(messages): migrate message thread create page to React #3176

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

Conversation

wescopeland
Copy link
Member

@wescopeland wescopeland commented Feb 5, 2025

This PR finalizes the messaging system's migration to React. In this branch, both the PHP UI and React UI for the new create page are present.

How to test this PR:

sail artisan migrate

http://localhost:64000/messages/create2
http://localhost:64000/messages/create

You should also test various template scenarios, ie:

http://localhost:64000/messages/create2?message=I%27d%20like%20to%20report%20an%20unwelcome%20concept%20in%20%5Bach%3D24%5D.%0A%20%20%0A-%20Which%20Unwelcome%20Concept%3A%0A%28Insert%20which%20concept%20from%20the%20docs%20here%29%0A%20%20%0A-%20Detailed%20Explanation%3A%0A%28Provide%20as%20much%20detail%20as%20possible%20here.%20Assume%20the%20reader%20may%20not%20have%20played%20the%20game%20before.%20The%20more%20detail%20you%20provide%2C%20the%20better%20your%20case.%29&subject=Unwelcome%20Concept%3A%20Bouncing%20Combo%20Master%20%5B24%5D%20%28Sonic%20the%20Hedgehog%29&templateKind=unwelcome-concept&to=DevCompliance

One important technical note is the use of a fulltext index for user searches. Each search query takes approximately 10ms at a result, down from 3,000ms when using LIKE.

Before
Screenshot 2025-02-04 at 7 02 18 PM

After
Screenshot 2025-02-04 at 7 02 12 PM

@wescopeland wescopeland requested a review from a team February 5, 2025 00:02
@wescopeland wescopeland added the status/stacked This PR is stacked on top of another PR. label Feb 5, 2025
@wescopeland wescopeland changed the title feat(messages): migrate message thread show page to React feat(messages): migrate message thread create page to React Feb 5, 2025
@wescopeland wescopeland removed the status/stacked This PR is stacked on top of another PR. label Feb 8, 2025
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.

1 participant