Skip to content

Instantly share code, notes, and snippets.

@ryanreh99
Last active September 2, 2020 06:23
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save ryanreh99/270a7a15e7980d2409562c5a7a085349 to your computer and use it in GitHub Desktop.
Save ryanreh99/270a7a15e7980d2409562c5a7a085349 to your computer and use it in GitHub Desktop.

Work Product: Google Summer of Code 2020 with Zulip

Link to all Pull Requests

Link to all merged commits

search: Resume project to integrate search pills.

Relevant PRs:

#14930, #15360, #15172, #15484 and #15507(open)

  • Updates the previously merged search pills related code with the new navbar re-design changes and fixes the bugs related to search pills and the pills widget.
  • Refactor and de-duplicate the search and search legacy code paths along with various UI UX improvements. Done by merging the search-related functions which were previously extracted as legacy functions, merging the CSS blocks and simplifies the two test files.

message view: Fix various message list's rare race conditions.

Relevant PRs:

#15065, #15680 and #15771

  • Preserves message list's integrity by displaying newly received messages in order. Implemented by discarding the rendering of the event and re-fetching on scroll to prevent a second race condition.
  • Fixes a re-rendering issue for locally echoed messages. Implemented by updating data synchronously while rerendering remains deferred.
  • Updates the UI when message resend acknowledgment is received from events path before requests success path.

message view: Add loading indicators and fix several bugs with the message feed.

Relevant PRs:

#15098, #15273, #15358, #15385, #15479, #15528, #15852, #16177

  • Display the new bottom loading indicator for fetching new messages and fix bugs with the message loading indicators. Required changing logic about decisions related to displaying indicators to each individual view.
  • Refactors logic and fixes bugs with multiple message feed notices to be displayed at the appropriate view.

stream settings: Add support for adding all members of another stream.

Relevant PRs:

#15859

  • We re-use our input pills widget to make it easier to mass subscribe users to a stream.
  • Includes relevant extractions and creation of new files for the stream pills feature along with updation to the existing user pills typeahead.
  • Also includes the creation of a new stream_edit.js unit test file.

stream settings: Re-design stream edit page.

Relevant PRs:

#15756(merged-partially), WIP - streameditbranch

  • Preparatory refactoring for the redesign of the stream settings page. Also includes the extraction and adding of the organizational settings save/discard widget.
  • Status: Work in Progress.

notifications: Topics I Follow.

Relevant PRs:

#14555(open)

  • Version 1: Adds a new notifications category for followed topics (topics where the user has posted before, future versions would give control to users to follow/un-follow topics).
  • Email, push notifications and wildcard mentions along the with UI and API changes have been implemented and Desktop and Audible notifications yet to be implemented.
  • Status: Currently on hold due to performance-related issues.

settings: Miscellaneous Bug Fixes

Relevant PRs:

#15239 #15696

  • Minor notification settings and stream settings related frontend bug fixes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment