Fix post views bugs #167

jiordiviera · 2024-10-26T15:59:21Z

Correction de l'incrémentation des vues d'un post à l'actualisation de la page, et aussi ajout d'une image de secours lorsque l'image est inexistant. et aussi résolution du problème qui subvenait après avoir commenté une discussion .

jiordiviera · 2024-10-29T11:20:31Z

@mckenziearts J'ai refait le PR, J'ai aussi remarqué lorsque je voulais faire un don que cela marque une erreur 500.

mckenziearts · 2024-10-30T07:28:46Z

app/Http/Controllers/ArticlesController.php

        $user = Auth::user();

-        views($article)->record();
+        views($article)->cooldown(now()->addHours(2))->record();


Que résoud cette action et quel comportement tu reçois et comment devrait-il normalement se comporter

views($article)->record();
la première ligne comptabilise chaque vue sans restriction, tandis que views($article)->cooldown(now()->addHours(2))->record(); impose une attente de 2 heures entre les enregistrements de vues pour le même article.

Si 50 personnes regardent l'article au bout de 10 minutes il n'enregistre pas les 50 records ? Ou tu dis qu'il va enregistrer les 50 records après 2h

Non c'est pour dire que un utilisateur ne pourrait voir le post deux fois de suite, genre sa vue sera mis en cache de tel sorte que si il voit un poste, en actualisant le navigateur, on considérera que il a déjà vu le post , la le nombre de vue ne sera pas incrémenté cela pendant les deux prochaines heures. donc la vue de post par cette utilisateur sera gelée.

mckenziearts · 2024-10-30T07:29:26Z

app/Http/Livewire/Discussions/AddComment.php

            ->send();

-        $this->reset();
+        $this->reset('body');


C'est la même chose. Le reset réinitialise toutes les variables du formulaire

mckenziearts · 2024-10-30T07:29:58Z

app/helpers.php

    }
 }
+
+if(! function_exists('getAvatar')) {


Pas nécessaire

Ici je voulais faire une illustration afin de simuler l'utilisation du onerror, ce n'était pas définitif.

Faut supprimer cette fonction du coup on ne garde pas du coup qui n'est pas utile

mckenziearts · 2024-10-30T07:31:17Z

resources/views/articles/show.blade.php

                                <h2 class="text-xs leading-5 tracking-wide uppercase text-skin-base">{{ __('Article suivant') }}</h2>
                                <div class="mt-3 flex items-start space-x-2">
-                                    <img class="h-10 w-10 object-cover shadow-lg rounded-md" src="{{ $next->getFirstMediaUrl('media') ?? asset('images/socialcard.png') }}" alt="{{ $next->slug }}">
+                                    <img class="h-10 w-10 object-cover shadow-lg rounded-md" src="{{ $next->getFirstMediaUrl('media') ?? asset('images/socialcard.png') }}" alt="{{ $next->slug }}" onerror="this.onerror=null;this.src='{{ getAvatar($next->title) }}'">


Le onerror c'est quoi son utilité vu que il y'a déjà une condition? C'est la condition qu'il faut revoir

Le onerror gère les erreurs de chargement d'image et utilise une image alternative si le chargement échoue, ce qui améliore l'expérience utilisateur en fournissant une option de secours. car j'ai remarque sur la plateforme des images n'ayant pas chargé donc n'existant pas, et la place c'est le title de l'image qui s'affichait et cela détruisait l'ui

C'est la condition qu'il faut corriger dans ce cas parce que si un article n'a pas d'image il y'a un placeholder normalement

mckenziearts

Une fois que tu auras effectué les retours je pourrai merge ta PR

mckenziearts · 2024-10-31T04:05:58Z

app/helpers.php

    }
 }
+
+if(! function_exists('getAvatar')) {


Faut supprimer cette fonction du coup on ne garde pas du coup qui n'est pas utile

mckenziearts · 2024-10-31T04:07:05Z

resources/views/articles/show.blade.php

                                <h2 class="text-xs leading-5 tracking-wide uppercase text-skin-base">{{ __('Article suivant') }}</h2>
                                <div class="mt-3 flex items-start space-x-2">
-                                    <img class="h-10 w-10 object-cover shadow-lg rounded-md" src="{{ $next->getFirstMediaUrl('media') ?? asset('images/socialcard.png') }}" alt="{{ $next->slug }}">
+                                    <img class="h-10 w-10 object-cover shadow-lg rounded-md" src="{{ $next->getFirstMediaUrl('media') ?? asset('images/socialcard.png') }}" alt="{{ $next->slug }}" onerror="this.onerror=null;this.src='{{ getAvatar($next->title) }}'">


C'est la condition qu'il faut corriger dans ce cas parce que si un article n'a pas d'image il y'a un placeholder normalement

mckenziearts · 2024-11-01T04:50:24Z

resources/views/livewire/articles/_form.blade.php

@jiordiviera ce que je dis souvent c'est que si un fichier n'entre pas dasn le scope de ta tache faudrait éviter de les modifier. Parce que tu modifies ce fichier pourtant ta feature n'a pas besoin de le faire.

mckenziearts · 2024-11-01T04:50:49Z

resources/views/components/forms/single-upload.blade.php

                    {{ __('PNG, JPG, GIF up to 1MB') }}
                </p>
-                <input @focus="focused = true" @blur="focused = false" class="sr-only" type="file" {{ $attributes }} />
+                <input @focus="focused = true" @blur="focused = false" class="sr-only" type="file" accept="image/jpg, image/jpeg, image/png" {{ $attributes }} />


Annule cette modification

mckenziearts · 2024-11-01T04:51:13Z

resources/views/components/articles/overview.blade.php

            <img class="object-cover shadow-lg rounded-lg group-hover:opacity-75"
                 src="{{ $article->getFirstMediaUrl('media') }}"
-                 alt="{{ $article->title }}"
+                 alt="{{ $article->title }}" onerror="this.src='{{ asset('images/socialcard.png') }}'"


Annule cette modification

mckenziearts · 2024-11-01T04:51:27Z

resources/views/components/articles/card.blade.php

    <a href="{{ route('articles.show', $article) }}" class="group">
        <div class="aspect-w-3 aspect-h-2">
-            <img class="object-cover shadow-lg rounded-lg group-hover:opacity-75" src="{{ $article->getFirstMediaUrl('media') }}" alt="{{ $article->title }}" onerror="this.onerror=null;this.src='{{ getAvatar($article->title) }}'"  />
+            <img class="object-cover shadow-lg rounded-lg group-hover:opacity-75" src="{{ $article->getFirstMediaUrl('media') }}" alt="{{ $article->title }}" onerror="this.onerror=null;this.src='{{ asset('images/socialcard.png') }}'"  />


Annule cette modification

mckenziearts · 2024-11-01T04:51:32Z

resources/views/components/articles/card-with-author.blade.php

        </p>
        <div class="relative h-96 overflow-hidden">
-            <img class="w-full h-full object-cover shadow-lg rounded-lg group-hover:opacity-75" src="{{ $article->getFirstMediaUrl('media') }}" alt="{{ $article->title }}" />
+            <img class="w-full h-full object-cover shadow-lg rounded-lg group-hover:opacity-75" src="{{ $article->getFirstMediaUrl('media') }}" alt="{{ $article->title }}" onerror="this.src='{{ asset('images/socialcard.png') }}'" />


Annule cette modification

mckenziearts · 2024-11-01T04:51:39Z

resources/views/articles/show.blade.php

                                <h2 class="text-xs leading-5 tracking-wide uppercase text-skin-base">{{ __('Article précédent') }}</h2>
                                <div class="mt-3 flex items-start space-x-2">
-                                    <img class="h-10 w-10 object-cover shadow-lg rounded-md" src="{{ $previous->getFirstMediaUrl('media') ?? asset('images/socialcard.png') }}" alt="{{ $previous->slug }}" onerror="this.onerror=null;this.src='{{ getAvatar($previous->title) }}'">
+                                    <img class="h-10 w-10 object-cover shadow-lg rounded-md" src="{{ $previous->getFirstMediaUrl('media') ?? asset('images/socialcard.png') }}" alt="{{ $previous->slug }}" onerror="this.onerror=null; this.src='{{ asset('images/socialcard.png') }}'">


Annule cette modification

mckenziearts · 2024-11-01T04:51:44Z

resources/views/articles/show.blade.php

                                <h2 class="text-xs leading-5 tracking-wide uppercase text-skin-base">{{ __('Article suivant') }}</h2>
                                <div class="mt-3 flex items-start space-x-2">
-                                    <img class="h-10 w-10 object-cover shadow-lg rounded-md" src="{{ $next->getFirstMediaUrl('media') ?? asset('images/socialcard.png') }}" alt="{{ $next->slug }}" onerror="this.onerror=null;this.src='{{ getAvatar($next->title) }}'">
+                                    <img class="h-10 w-10 object-cover shadow-lg rounded-md" src="{{ $next->getFirstMediaUrl('media') ?? asset('images/socialcard.png') }}" alt="{{ $next->slug }}" onerror="this.onerror=null;this.src='{{ asset('images/socialcard.png') }}'">


Annule cette modification

mckenziearts · 2024-11-01T04:52:09Z

resources/views/articles/show.blade.php

                            @if($article->isNotApproved())
                                @hasanyrole('admin|moderator')
-                                    <button onclick="Livewire.emit('openModal', 'modals.approved-article', {{ json_encode([$article->id]) }})" type="button" class="-ml-px relative inline-flex items-center px-4 py-2 border border-skin-base bg-skin-card text-sm font-medium text-green-500 hover:bg-skin-card-muted focus:z-10 focus:outline-none focus:ring-1 focus:ring-green-500 focus:border-green-500 focus:border-green-500 focus:ring-offset-body">
+                                    <button onclick="Livewire.emit('openModal', 'modals.approved-article', {{ json_encode([$article->id]) }})" type="button" class="-ml-px relative inline-flex items-center px-4 py-2 border border-skin-base bg-skin-card text-sm font-medium text-green-500 hover:bg-skin-card-muted focus:z-10 focus:outline-none focus:ring-1 focus:ring-green-500 focus:border-green-500 focus:ring-offset-body">


rétabli les classes que tu as retiré

mckenziearts · 2024-11-01T04:52:16Z

resources/views/articles/show.blade.php

                                <h2 class="text-xs leading-5 font-medium tracking-wide uppercase text-skin-base">Article précédent</h2>
                                <a href="{{ route('articles.show', $previous) }}" class="mt-3 flex items-start space-x-2">
-                                    <img class="h-8 w-8 object-cover shadow-lg rounded-md" src="{{ $previous->getFirstMediaUrl('media') }}" alt="{{ $previous->slug }}">
+                                    <img class="h-8 w-8 object-cover shadow-lg rounded-md" src="{{ $previous->getFirstMediaUrl('media') }}" alt="{{ $previous->slug }}" onerror="this.onerror=null;this.src='{{ asset('images/socialcard.png') }}'">


Annule cette modification

jiordiviera added 3 commits October 26, 2024 15:54

initialize branch

8a9ae6e

correction de l'erreur 500 afficher apres avoir commenté une discussion

5858a64

correction des problèmes avec le merge

38b996e

jiordiviera closed this Oct 29, 2024

jiordiviera reopened this Oct 29, 2024

mckenziearts requested changes Oct 30, 2024

View reviewed changes

mckenziearts requested changes Oct 31, 2024

View reviewed changes

file preview fix

002b161

mckenziearts requested changes Nov 1, 2024

View reviewed changes

correction du merge

a8324e2

mckenziearts approved these changes Nov 1, 2024

View reviewed changes

mckenziearts merged commit 84492f7 into laravelcm:main Nov 1, 2024
3 of 5 checks passed

Fix post views bugs #167

Fix post views bugs #167

Conversation

jiordiviera commented Oct 26, 2024

Uh oh!

jiordiviera commented Oct 29, 2024

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mckenziearts left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants