Inspiration
This week provided us with an opportunity to look through all the projects in detail and contribute to those which we found exciting! The opportunity to give back to the community by adding features, fixing bugs, or adding documentation to projects built by Fellows was exciting!
Our contributions
Sociomark #108
https://github.com/MLH-Fellowship/SocioMark/pull/128
The issue said to display the user has no posts when len(posts) = 0. Since we could play with react and tailwind CSS , we took this issue. After doing the same, added a feature to create the first post if the user wanted to and when the user goes to other profiles with no posts, only "no posts" would be displayed.
Sociomark #106
https://github.com/MLH-Fellowship/SocioMark/pull/136
Currently, the project called /post/all to get all the posts in the database - we need to return back only those posts whose post.report_counter < 5, where 5 is just a threshold.That was the issue. Since we could get hands-on experience on fastapi we took this issue. What we did is
- Return the post only if the number of reports < threshold in these APIs
- retrieve_posts API (Retrieve all posts with user id present in the database)
- retrieve_all_posts API (Retrieve all posts from the database)
- retrieve_post API (Retrieve a post with a matching ID)
- Added threshold to an env so that developer can change the variable if he wants to.
Sociomark #109
https://github.com/MLH-Fellowship/SocioMark/pull/124
The issue was the placeholder text needs to be reset every time the user submits the form (Applicable for every input in frontend). Again to play with the react frontend we took up this issue and implemented the same. The values in forms or input fields in: Login, Register, Search bar, Add posts are now all cleared after the user clicks on "Submit" or chooses a user profile
Linker #33
https://github.com/MyrrhDev/Linker/pull/33
In this project initially from the documentation a developer would never know how to run the extension locally. Therefore took up this issue and added an update.
Sociomark #142
https://github.com/MLH-Fellowship/SocioMark/pull/143
This was an issue we took up in the category of "bug fix". This was a bug identified by ourselves You need to force the "int()" on that otherwise it raises a "comparison error int and string" and does not load any posts. So we changed type of threshold number to fix "comparison between int() and str()" error.
Sociomark #112
https://github.com/MLH-Fellowship/SocioMark/pull/144
Currently, when you click on the Verify button, it shows the user_id, name and email_id. We need to convert it to a link and that was the issue. So now when you click on the Verify button, the modal now links to the original author's profile
Challenges we ran into
- Whether we could fix the issue or not, although we were able to make it in the end.
Accomplishments that we're proud of
- In a short time, we could make a good number of PRs
- Learnt fastapi and played with tailwind CSS
What we learned
- How to contribute effectively to open source projects
- Communication with the maintainers
- Tech stack like fastapi and react
Log in or sign up for Devpost to join the conversation.