For this week, we have officially begun our Hacktoberfest event! We were tasked with finding issues to work on with the tag Hacktoberfest.
After rigorous searching for a proper issue to work on, I found Nocodb. This repository posted this issue, and I challenged myself to take it.
After reading through the codebase and trying to follow the installing instructions, I was able to get the open source project to successfully execute. A big challenge for me was trying to understand the flow of the code. Understanding what each file did, and how it all came together.
Once I figured out the code, I began my journey of fixing the bug. The issue brought up that the email field was using the text type, instead of an email type. This would make the input be treated as case sensitive. Instead, the proper way is to have the email field be case insensitive. The author of the issue mentioned that when he entered the email, after typing in a ".", the first letter of the input would be capitalized. When I was trying to reproduce the bug, I was using Chrome, Firefox and Microsoft Edge, and was unable to get the first letter of the email field to capitalize. I believe that this issue is only on Safari, but since I am on a Windows machine, I am unable to test it. Regardless, changing the field to a email type, instead of a text type would most likely resolve this issue.
The authors created steps for Licensing and modifying an existing file. I followed these steps and was able to create a PR with my changes.
I have been able to successfully merge my changes, which could be found in this PR PR Link
Top comments (0)