- Dlaczego wykorzystałeś Mongo? Wszytko tutaj brzmi na dane relacyjne.
- Dlaczego w pętli co 100ms wykonujesz zapytanie na
/api/admin_dashboard/auth? Gdybyś miał na stronie kilkuset użytkowników jednocześnie, mógłbyś powiedzieć pa, pa, serwer.
- Rozmiar fontów w panelu administracyjnym jest IMO znacznie, znacznie za duży.
-
https://damp-badlands-92873.herokuapp.com/admin_dashboard - komunikat no access brzmi jak gdyby został sklecony na kolanie.
- Po wpisaniu nieprawidłowych danych w formularzu logowania do panelu administracyjnego nie jest pokazywany żaden komunikat.
- Dla czytelności, trzymaj się jednego ficzera per strona - zamiast
/admin_dashboard służącego do zarządzaniu wszystkim, zrób /admin/messages z tabelą pozwalającą na wyszukiwanie oraz usuwanie wiadomości, i osobno /admin/messages/create służącego do tworzenia wiadomości.
- Spróbuj usunąć jakiś plik na Twoim komputerze - o ile nie zmieniłeś domyślnej konfiguracji, powinien Ci się pokazać komunikat
czy na pewno chcesz usunąć ten plik?; dodaj ten sam komunikat do przycisku Usuń wiadomość. Generalnie: każda destruktywna akcja powinna być za potwierdzeniem.
- Po naciśnięciu przycisku
Dodaj wiadomość nic się nie dzieje - nie otrzymuję żadnej informacji czy coś zostało dodane, czy może wystąpił błąd.
- Będąc w panelu administracyjnym nie jestem w stanie myszką wrócić na stronę główną, muszę ręcznie zmieniać URL.
- Brakuje podstrony
404 - pokazuje się w tytule strony heh.
- Spójrz na stronę główną i zastanów się, w jaki sposób mógłbyś zaprezentować te dane tak, abyś nie musiał pisać
tytuł ogłoszenia:, tekst ogłoszenia: etc. Zauważ, że np. tutaj na forum nie masz etykietek autor posta:, treść posta:, data utworzenia posta:, a mimo to hierarchia i odpowiednie ułożenie elementów strony sprawia, że potrafisz odpowiednio zinterpretować to, co widzisz.
Btw, zdaje się, że nie zacommitowałeś zmian do GitHuba, ponieważ tam znajduje się sam domyślny szkielet aplikacji.
Wprowadziłem kilka zmian w moim projekcie. Miałem ostatnio mało czasu, długo to trwało. Dziękuję, że zechciałeś zapoznać się z moją pracą. Odpowiadam na Twoje uwagi:
- Z kilku powodów. Przeglądając oferty pracy dla junior front-end developerów, spośród wymagań dotyczących baz danych najczęściej pojawiała się właśnie Mongo. Przeglądając kanały kilku YouTuberów, często widziałem projekty robione w MERN stacku. Przez oba te fakty pomyślałem, że Mongo to z jakiegoś powodu najlepszy lub najmodniejszy obecnie wybór. To również pierwsza i jedyna do tej pory baza danych, jakiej używałem, od czegoś musiałem zacząć. Twoje pytanie skłoniło mnie do googlowania i znalazłem ten artykuł. W podsumowaniu natrafiłem na takie zdanie "MongoDB stores key names for each value pairs. Also, due to no functionality of joins, there is data redundancy. This results in increasing unnecessary usage of memory." Czy dlatego powinienem był użyć bazy SQL'owej?
- Prawdopodobnie dlatego, że umieściłem to zapytanie w render(). Przerzuciłem je do componentWillMount().
- Zmieniłem na mniejsze.
- Zmieniłem na "Nie masz uprawnień do przeglądania tej strony". Nie wiem, czy chodziło Ci o samą treść komunikatu, czy o to, że powinien być on ładniej ostylowany.
- Dodałem komunikat.
- Zrobiłem tak jak mówiłeś, faktycznie lepiej się to prezentuje i użytkowanie wydaje mi się "milsze".
- Dodałem potwierdzanie usuwania.
- Dodałem komunikat dodania wiadomości. Zablokowałem możliwość dodawania słów dłuższych niż 32 znaki w tytule i dodania pustego newsa.
- Zrobiłem przycisk.
- Dodałem komunikat błędu z przyciskiem przekierowującym na stronę główną.
- Zrobiłem homepage z wyświetlaniem newsów nie myśląc kompletnie o tym jak to będzie wyglądało, bo założyłem, że korzystając z tego panelu w jakimś rzeczywistym projekcie i tak ostyluję wyświetlane newsy zgodnie ze stylem tworzonej strony. Chciałem, żeby działała tylko logika. Myślę jednak, że masz rację i nie powinienem w żadnym projekcie zostawiać takich brzydkich elementów, dlatego ostylowałem homepage, żeby "jakoś" wyglądała.
Co do ostatniej Twojej uwagi - zacommitowałem wszystko do Githuba, ale na brancha add_news_pl_heroku. Czy powinienem wrzucić wszystko również na "mastera"? Nie wiem jakie panują w tej kwestii konwencje. Sądząc po tym, że widziałeś sam domyślny szkielet, mniemam, że widziałeś kod z "mastera".
Po drodze zauważyłem, że mój kod powoduje pewien błąd. Sprawiał on, że po usunięciu danego newsa, **nie zawsze ** znikał on z panelu usuwania bez ponownego przerenderowania strony. Poradziłem sobie z tym tworząc ifa sprawdzającego czy długość tablicy listy newsów znajdującej się w state jest równa długości listy newsów otrzymanej z bazy danych poprzez backend. Jeśli tak, to znaczy, że news się nie usunął i w takim przypadku ustawiłem ponowne wykonanie "geta". Nie wiem co jest przyczyną tego błędu, być może połączenie na linii frontend - backend - baza danych - backend - frontend "nie nadąża" przed przerenderowaniem strony powodowanym przez wciśnięcie przycisku "usuń".
Wspomnę jeszcze o weryfikacji wpisywanego hasła przy logowaniu. Zrobiłem to tak, że Axios w Reakcie wysyła żądanie "post" do backendu, który sprawdza zgodność wpisanego hasła z prawidłowym hasłem zapisanym w bazie danych. Następnie backend odpowiada frontendowi, czy hasło jest w porządku. Jeśli tak, to frontend wysyła kolejne żądanie "post", tym razem do route "admin_dashboard". W pliku tego route odbiór rządania "post" ustawia zmienną "auth" na true. Zmienna ta jest umieszczona również w rządaniu "get" w tym samym pliku route, które odbierane jest przez komponent "Page", w którym zagnieżdżony jest komponent panelu administratora. "Page" przesyła w propsie odebranego geta do panelu. Jeśli get = true, to strona panelu się renderuje. Próbowałem zrobić ten system tak, żeby nie trzeba było wysyłać drugiego rządania do backendu. Zamiast tego chciałem zrobić weryfikację poprzez state w komponencie matce - "Page". Niestety nie dałem rady sprostać temu wyzwaniu. Zrobiłem funkcję w komponencie matce, przekazałem ją w propsie do dziecka i w dziecku wywołełem, tak żeby przez dziecko zmienić state w matce, ale to nie działało.
Zmieniłem adres heroku: LINK
Będę bardzo wdzięczny za każde kolejne komentarze i uwagi. Przypominam, że to moja pierwsza styczność zarówno z Reaktem, Nodem, Mongo jak i Heroku. Jestem również całkowitym samoukiem, bez kontaktu z żadnym programistą. Proszę o wyrozumiałość i cierpliwość. :D