-
Notifications
You must be signed in to change notification settings - Fork 0
Sprint 9
This week is a return to one week sprint and was kind of weird to come back to more small task. Everything went well and time estimated were cool. Working with Emilien on end-to-end encryption s always a good time. the only difficulty I had is when to merge the design-refont which was separated from the main from 4 weeks was king of a hard time.
My task this week was to address the majority of the comments in the code review. This was maybe not the most interesting task but was definitely necessary. This task also required some collaboration with the rest of the team since it was necessary that I merge after everyone but this went well. The time estimates went quite well since I had already done more than I expected within the time estimate.
My task this week took a lot longer than expected, since I overestimated Firebase's forum ability to be mocked. Also, there were a lot of unexpected bugs in our codebase (forum & database) that had to be fixed in order to achieve my task. However, I'm happy with the result, the coverage increased a bit (even though I only adapted the existing tests, not even adding new ones yet!).
I still have to mock the forum listener, which I will do next sprint. Also, now the cache integration can finally be done and tested without blindness.
This week I worked together with Adrien on the end-2-end encryption feature, as well as implemented an option for the users to use the app as guest, without signing in. I am happy with how things went this week, the refactoring is making the code base very easy and pleasant to work with and the app looks awesome :D. I'm looking forward to the last sprint so that we can have a finished product to make our TAs proud in the competition.
This sprint was more about making the app fully functional, so I took the task to link all the missing buttons with their functionalities. This includes implementing the rating feature which enables saving users' feedback on our database. On top of that, I implemented the direct sign-in to the app independently from Firebase, which is important in order to have a fully offline mode. Thankfully, I didn't encounter any big problems during this sprint.
It was really fun implementing the rating feature, so sad the end is approaching really fast :(
During this sprint, I worked on 3 separate tasks. The most important one was figuring out the local storage bug which was a bit time-consuming but it's all good now. I also worked on the post notifications. Previously, when a professional accepted to receive notifications from a specific forum category, he would receive a notification for every post and every reply to every post in that category. This is now fixed by differentiating between a post and a reply to a post. Moreover, when a user makes a new post he can now specify whether he wants to receive notifications when someone replies to his post or not.
I also added the post time to forum posts.
The team was incredible this week. We had a lot of constructive meetings and are very happy about the progress we achieved together. We discussed about the tasks that remain for the next sprint, and concluded that we will do everything to put back the coverage into better territory. We have an artificially low coverage because we had to remove entire files that failed some time on Cirrus only and did not want to relaunch Cirrus 10 times every time.
Next sprint, we will tackle this issue and achieve all-time-high coverage results! Mark my words!