apartresearch / aisafetyideas

💡 The web app CI/CD for aisafetyideas.com
https://aisafetyideas.com
8 stars 3 forks source link

Fix issue #26 #29

Closed AlexTMjugador closed 1 year ago

AlexTMjugador commented 1 year ago

After locally setting up the project and debugging it, I came to the conclusion that the exception described in the aforementioned issue was caused by a referential integrity constraint violation; namely, that there was at least one comment that referred to an unexisting author.

I didn't find any reason why this issue would only affect Firefox, so I think that the best fix would be to locate such troublesome comments in the database and either edit their author information or delete them. However, to prevent these problems from bringing down the site in the future, let's ignore comments with unknown authors in the frontend, which I think works fine.

vercel[bot] commented 1 year ago

@AlexTMjugador is attempting to deploy a commit to the Apart Research Team on Vercel.

A member of the Team first needs to authorize it.

AlexTMjugador commented 1 year ago

This probably fixes #28 as well. It's likely that different browsers show different messages for the same error here.

Edit: by the same token, #27 is also probably fixed with this!

vercel[bot] commented 1 year ago

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

Name Status Preview Comments Updated (UTC)
aisafetyideas ❌ Failed (Inspect) Jun 17, 2023 0:35am