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

Hotfix for critical bug 1043 #1050

Merged
merged 2 commits into from
Dec 12, 2024
Merged

Hotfix for critical bug 1043 #1050

merged 2 commits into from
Dec 12, 2024

Conversation

CodeWritingCow
Copy link
Collaborator

@CodeWritingCow CodeWritingCow commented Dec 12, 2024

Description

This pull request is a hotfix for #1043, a critical bug that crashes the website after a user searches a property.

This PR fixes the bug by reverting PR #1003, and part of another, #1008

Screen recording

hotfix.pr.1050.mov

Copy link

vercel bot commented Dec 12, 2024

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

Name Status Preview Comments Updated (UTC)
vacant-lots-proj ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 12, 2024 7:08am

@github-actions github-actions bot requested a review from zigouras December 12, 2024 07:07
@CodeWritingCow CodeWritingCow removed the request for review from zigouras December 12, 2024 07:08
@CodeWritingCow CodeWritingCow changed the title Revert 1003 Hotfix for bug 1043 Dec 12, 2024
@CodeWritingCow CodeWritingCow changed the title Hotfix for bug 1043 Hotfix for critical bug 1043 Dec 12, 2024
@CodeWritingCow CodeWritingCow merged commit 721d0b4 into main Dec 12, 2024
10 checks passed
@CodeWritingCow
Copy link
Collaborator Author

The normal PR process for this team is to submit a PR against staging branch first, then create a PR from staging to main.

However, I submitted this hotfix PR against main and skipped staging. That's because staging currently has code that haven't been deployed to main yet.

I didn't want to "muddy the water" and submit a PR from staging with unrelated code. I wanted to keep a separation of concerns: hotfix should go in its own PR and not be mixed up with unrelated code. Moreover, I wanted to avoid the possibility of introducing any potential bug in staging to main/production.

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.

1 participant