Skip to content

Conversation

@Asheraf
Copy link
Contributor

@Asheraf Asheraf commented Dec 19, 2018

Pull Request Prelude

TODO

  • Blocking state when ui is open

@HerculesWSAPI
Copy link
Contributor

This change is Reviewable

@Asheraf Asheraf added the status:inprogress Issue is being worked on / the pull request is still a WIP label Dec 19, 2018
@Asheraf Asheraf added this to the Release v2019.01.13 milestone Dec 19, 2018
@HerculesWSAPI
Copy link
Contributor

This pull request introduces 1 alert when merging 98a38e3 into 406ae36 - view on LGTM.com

new alerts:

  • 1 for FIXME comment

Comment posted by LGTM.com

@HerculesWSAPI
Copy link
Contributor

This pull request introduces 1 alert when merging 5bff04d into 406ae36 - view on LGTM.com

new alerts:

  • 1 for FIXME comment

Comment posted by LGTM.com

@hemagx
Copy link
Contributor

hemagx commented Mar 4, 2019

Also finally, please use enum for all client responses.

@4144
Copy link
Contributor

4144 commented Mar 21, 2019

i hide some useless review comments, but some useless comments probably still left

@MishimaHaruna MishimaHaruna removed this from the Release v2019.04.07 milestone Apr 7, 2019
@Asheraf Asheraf force-pushed the lapineddukddak branch 3 times, most recently from 5c4167a to d6dd79b Compare May 23, 2019 03:15
@Asheraf Asheraf added this to the Release v2019.06.30 milestone Jun 2, 2019
@MishimaHaruna MishimaHaruna removed this from the Release v2019.07.28 milestone Jul 28, 2019
@Asheraf Asheraf removed the status:inprogress Issue is being worked on / the pull request is still a WIP label Aug 24, 2019
@Asheraf Asheraf added the status:code-review Awaiting code review label Aug 24, 2019
@Asheraf Asheraf force-pushed the lapineddukddak branch 2 times, most recently from ff08b34 to a2b9dfe Compare August 24, 2019 22:18
@Asheraf Asheraf force-pushed the lapineddukddak branch 4 times, most recently from 7409b48 to d1bf9ec Compare August 26, 2019 02:08
@4144 4144 added this to the Release v2019.09.22 milestone Sep 22, 2019
@MishimaHaruna MishimaHaruna merged commit 1132e2e into HerculesWS:master Sep 23, 2019
@Asheraf Asheraf deleted the lapineddukddak branch June 2, 2022 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code-review Awaiting code review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants