Posts

End of Open Source Course

Image
Last issue for OSD600 This was the last week for open source course. Tony and I fixed one more issue for it. Issue: #468 Add more test cases to sanitize-html.test.js Pull request: #486 Add test cases for sanitize-html In order to fix this issue, I spent some time to understand the sanitize-html . Sanitize-html is used to clean up user-submitted HTML, it will remove unwanted CSS. They have some default tags as below. Conclusion of OSD600 Time ran so fast, this week is the last week of this semester. I checked all the posts I wrote from the beginning of this semester. I wrote more than 10 blogs for this course in this semester.  Remember the first class of this course. I didn't know what is open source. After the introduction of this course, I was scared cause this course we need to fix the publish issues. I don't think I can do it. I am not very good in programming, I didn't do any project by myself. All the experience I had at that time was from the assignm

OSD600 4.0 Telescope update

Image
In the week, I fixed one external issue and keep working on the internal issue. External: AncientBeast  is a website game that you can play on you laptop and your phone I fixed an issue for UI from a game. Below is how the page looks like in website. As we can see, there is padding in left and right side. Here is the web looks like in phone. As we can see, the is no left and right padding. As the member for this project, he/she wants the page looks the same whatever it's in desktop or phone. Issue: No div side padding Pull request: Fix issue#97 removed padding Conclusion: This project is using php to build it. They use nvm (Node Version Manager). When I decided to fix this issue, I was thinking it was not different issue. Just change padding to 0 should be work. However, after I stepped into the project, problem came out. They use iframe. I cannot just remove padding cause they didn't set up the padding, They limit the size of the iframe. I need to rest the wid

OSD600 4.0 Telescope contribute

Image
In the 3.0 I created a function to count blog posts from different domain and created tester to test it collaborate with my classmate.  Here is what we did in 3.0. In the 4.0, we decided to work together again. We picked two issue from the list. Issue 1: #260 Allow a Post to get stored in Redis. Issue2: #261 Update frontend design to include all Post data. To finish the 4.0, one internal pull request has to be merged and fixed one external issue. we have around three weeks to finish it. So we decided to focus on the fist issue in this week. Pull request 1: #332 Fix #260 implement addPost and getPost method I checked the code my professor created for getFeed and getFeeds method in storage.js. I was so surprised  that he only use around 10 lines to do it. I need to understand what this code is doing.  To understand his code, I have know what the method is it. Even though I discussed it with my collaborator, we didn't fully understand it. However, my professor explained it

OSD600 3.0 Telescopen update

Image
As I mention in the previous blog, we are going to contribute to telescope.  Click here to see the GitHub. I checked the issues that we have so far and cannot find any issues that I can work on. So I discussed with my classmate and create the issues for us. After our discussion, We decided to make a function to count how many blog come from different domain. Issue --   Implement blog post count by domain .  Click here to check the pull request. Issue -- Test the function of counting each domain post . Click  here  to check the pull request. Issue -- Updarte blogDomainCounter to pass tester Click  here  to check the pull request. When we discussed to make the function to count it, we though it should be done soon cause it was not so difficult. However, when we actually open a file and started working on it. we met so many problem such as: 1. How we can get the urls. 2. How to get the domain. I read all the code we had so far and understand what other people  did so f

OSD600 Release 3.0 Telescope

Image
Introduction Telescope is a tool for tracking Seneca's open source involvement.  However, this website was built long time ago. Click here then you can visit the telescope we are using so far. For the release 3.0, we are going to refactor the website together with 50+ students using the popular tool and language. When I heard that I was so exited cause I have chance to use the knowledge I learn so far. However, after the first class of discuss what we should use to build it. I felt so confuse.  I don't even know how I can step into the project.  Here is the telescope GitHub.  I forked and cloned the project for the first step. You can check my Release1.0 , there is a detail instruction of how I started working on a project.  Prerequisites: Node.js (npm) Redis is an in-memory data structure store but I have never heard it before. I decided to learn it first.  Download Redis from here . And install it follow the installation. Problem I forked the project many d

The last pr for Hacktoberfest

Image
The last four days for Hacktoberfest including today. For the last pr, I fixed an issue on css. When I found the issue, I though it was an easy issue to fix. However, after I stepped into it. I found some problem. Set up the environment always cost me so much time. The project is using React that I am not good at it. This is a thinking app that already launch. Click here go to the website. When I checked the GitHub, there are 70+ issues. I picked the issue of fix bullet positioning. It opened 12 says ago. Click here i s the issue. Click here for the pull request. I finished all the pull request Hacktoberfest need. I am so happy. When the first time I heard Hacktoberfest from my professor. I was worried about I cannot finish it. I was thinking fixing issues was very hard. I was not confident in it. Finally I did it. I learned a lot during fixing the issue. Even though a small issue, I can learn something new from the small issue as well. I am very enjoy it.

3rd pull request for Hacktoberfest

Image
Here is the latest  pull request I did. Rename "child" to thought" Here is the issue: Here is the pull request: Conclusion: When I found this issue, I think it's very easy and I can fix it very soon. However, after I fixed it. The problem came out. When I change the "child", I forgot to do the change for export and import, the project cannot run. I need to google it how to solve this problem. Ptrvious:  2nd pull request for Hacktoberfest