michalbe / jsnews-client

:alien: Commandline client for JS News
http://github.com/michalbe/jsnews-client
MIT License
38 stars 5 forks source link

Trochę zmian #31

Closed chemikpil closed 10 years ago

chemikpil commented 10 years ago

Posiedziałem w weekend i trochę pozmieniałem. Nie obraź się, że tak dużo :)

Co zrobiłem?

To chyba tyle. Do zrobienia zostało jeszcze:

michalbe commented 10 years ago

Extra, wezme się za review dziś. W międzyczasie prosiłbym o dwuspacjowe indenty jak w poważnej internetowej aplikacji :).

Zastanawia mnie wyłączenie notyfikacji o komentarzach itp. To akurat był jeden z featurów przez który ta aplikacja w ogole powstała. Może moglibyśmy ustawić flagę w profilu, coś w stylu notifyAboutAll która jeśli jest true to działa jak pierwotna wersja, a jeśli false to uaktywnia followowanie postów (których ID i ostatni stan - ilość komci i lajków - raczej bez większej gimnastyki mogą być keszowane nawet w pliku tekstowym w katalogu aplikcji, wtedy stan followowanych zachowa sie także po restarcie). Co myślisz?

chemikpil commented 10 years ago

Zmieniłem wcięcia na "poważne" :). Poprawiłem też to co znalazłeś dotychczas.

Co do komentarzy. Ogólnie to nie problem by zrobić taką flagę. Myslałem by dla każdej grupy taką dodać w configu. Zapisywanie w pliku :+1:

Trzeba by jeszcze rozkminić notyfikacje tak, by klikniecie w nią otwierało dany post. Teraz problem jest taki, że jak obserwowany post bedzie np. 9 z kolei to notyfikacja przyjdzie, ale my go nie zobaczymy w terminalu. Mega słabe.

michalbe commented 10 years ago

Otwórz na to Issue githubowe,ja w miedzycasie reviewuje Twój PR.

michalbe commented 10 years ago

Dziwna rzecz - skomentowałem post i dostałem notyfikację że skomentował go autor postu, nie ja. screen shot 2014-06-23 at 11 58 43

michalbe commented 10 years ago

Cała reszta działa miodnie.

chemikpil commented 10 years ago

Haha, ale dałem ciała z tym powiadomieniem. Faktycznie zwraca ownera bo tak mu przekazałem. Jak znajdę chwile to poprawię.

Issue też dodam przy wolnej chwili.

michalbe commented 10 years ago

Dzięki, jesteś szefem!