mobilecodereviewer / MobileCodeReviewer

0 stars 0 forks source link

[M] Brak możliwości dodawania komentarzy przy przełączaniu metody autentykacji #5

Closed fracz closed 9 years ago

fracz commented 9 years ago

Próbowałem dwuklik (dwutap), long tap itp - nic nie działa. Na screenie w podręczniku jest gwiazdka - nie ma u mnie takiego przycisku.

d00d171 commented 9 years ago

Byłeś zalogowany? Gwiazdka pojawia się tylko kiedy na ekranie konfiguracji podasz HTTP Credentials

fracz commented 9 years ago

Owszem - jestem zalogowany.

d00d171 commented 9 years ago

Jeśli chodzi o komentarz do całej zmiany to gwiazdka powinna być widoczna na ekranie ze szczegółami zmiany (Change Details) dla zalogowanego użytkownika. Żeby skomentować linijkę w kodzie źródłowym trzeba zrobić longclicka na wybranej linii.

Mógłbyś jeszcze raz się upewnić że obie te funkcjonalności nie działąją? Czy zmieniałeś w międzyczasie login/hasło/urla na pierwszym ekranie?

fracz commented 9 years ago

Tak - zmieniałem (w sensie w aplikacji z anonymous na authenticated). Po zmianie na authenticated i restarcie aplikacji zaczęło działać.

d00d171 commented 9 years ago

Błąd już nie występuje kiedy się przełączasz?

fracz commented 9 years ago

Nie sprawdzałem :-)

fracz commented 9 years ago

Nadal nie działa. Kroki:

  1. Świeża instalacja aplikacji
  2. Wprowadzenie danych repo bez użytkownika
  3. Pokazanie listy zmian
  4. Powrót do konfiguracji repo -> podanie użytkownika & apply
  5. Zmiany wyświetlają się, ale po wejściu do nich nie ma przycisku "gwiazdka"

PS. Nie musicie wgrywać za każym razem zbudowanej apki - mogę sobie pullować kod i budować ją u siebie - ./gradlew installDebug przechodzi bez problemu. Rozumiem, że pracujecie na branchu dev?

d00d171 commented 9 years ago

Tak, działamy na devie. Dla nas byłoby to znacznie wygodniejsze rozwiązanie gdybyś mógł to u siebie budować. Jeżeli nie stanowiłoby to problemu to moglibyśmy tak ustalić. Zbudowaną apkę na wiki będziemy ewentualnie wrzucać po jakichś większych zmianach / co jakiś czas / przy kolejnym releasie. Co ty na to ?

fracz commented 9 years ago

Dla mnie to żaden problem. Zauważyłem tylko, że na ./gradlew build nie wszystkie testy przechodzą :-)

d00d171 commented 9 years ago

To jest zamierzone zachowanie.

d00d171 commented 9 years ago

Poprawione w wersji 0.3.10