Skip to content

Instantly share code, notes, and snippets.

@thedeveloperr
Last active August 31, 2020 16:15
Show Gist options
  • Save thedeveloperr/0e75649034e103efe7549d96ca09f9e3 to your computer and use it in GitHub Desktop.
Save thedeveloperr/0e75649034e103efe7549d96ca09f9e3 to your computer and use it in GitHub Desktop.
This Gist contains work done for Zulip Open Source Project's Team chat software during Google summer of code 2020

Work Product Document for Zulip open source project as GSoC 2020 student developer.

Mohit Gupta | Github

Areas worked in: Tests, Search, Refactor, Keyboard support

Introduction

This gist contains work done during Google Summer of Code 2020 with Zulip open source project. Most of my work is fullstack. I Worked both on:

  • Zulip's main frontend webapp which is in Javascript
  • Zulip backend server which uses Python

Work have been focused upon:

  • Refactor of code related to tests
  • Cleaning up logging output and verifying it without printing in test output making CI test output clean
  • Adding script to detect any unnecessary console output while running tests in CI
  • Adding keyboard support for left and right sidebar popover menu.
  • Making has:link, has:attachment, has:image work on client side.
  • Fixing full text search bug.

I want to thank my GSoC's mentor Shubham Padia for guidance. I also want to thank Zulip's Tim Abbott, Steve Howell, Anders Kaseorg, Rohitt Vashishtha, and many other community members for their support and help during my GSoC journey. I am grateful.

Work summary

Issue: Can only find URLs when entering them completely. #9165 Right now user have to search messages by typing full urls and the postgres don't index parts of urls. Aim of the work is to allow user to search by typing parts of url instead of full url.

PRs that solves it:

  • [WIP] search: Allow searching of messages with link by typing parts of url. It's a WIP work. Essentially it does is that extract urls from rendered html of messages and tokenise them by replacing characters: .,-,_,/ in the url with space and then index them in postgres for full text search. This will allow the user to search the rendered content by typing just parts of url instead of typing full urls

PRs that solves it:

PR that solves it:

PRs that solves it:

  • popovers: Improve keyboard navigation #15722 This PR adds ability to move the arrow buttons to navigate the items of of the left and right sidebar's popover menus when it is open. This PR also had a commit to open stream popover menu using 'i' hotkey but the commit was removed by a force push as more planning was needed for right way to open the stream popover with keyboard.

PRs that solves it:

Issue: Refactor and split the tests test_messages.py into seperate files.

PRs that do the refactors:

Issue: Remove word bugdown and rename it with word markdown.

PRs that do the refactors:

  • zulip/zulip#15569, zulip/zulip#15606 Zulip's markdown flavor for historical reasons was called bugdown which was a confusing term. So a refactor was done to change the word bugdown to markdown from whole code. The challenge like any other refactor was to do this without breaking anything.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment