Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zmiany dla notyfikacji, konwersacji i prywatnych wiadomości (do dyskusji) #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

noisy
Copy link
Contributor

@noisy noisy commented May 17, 2014

Ten pull request raczej nie jest gotowy do wciagnięcia (tzn. wszystko powinno działać, ale nie wszystko podoba mi się pod względem tym jak to jest zrobione).

Pull request założyłem, by to przedyskutować to jak to powinno być rozwijane dalej.

Generalnie jakiś czas temu zrobiłem te zmiany, by móc w łatwy sposób zczytywać botem notyfikacje i odpisywać na jakikolwiek typ wiadomości (wiadomość prywatna/komentarz do wpisu na mikro/komentarz pod znaleziskiem).

Funkcje reply, które dodałem do odpowiednich obiektów zwracanych przez funkcje takie jak get_link_comments, get_entry pozwalają na poziomie notyfikacji nie martwić się, skąd notyfikacja przyszła i po prostu zrobić na niej reply. Natomiast to nie do końca podoba mi się to w połączeniu z prostymi typami dotychczas zwracanymi przez wszystkie metody.

Stąd pomysł na https://github.com/p1c2u/wykop-sdk/issues/6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant