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.

Sign up for GitHub

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

Jump to bottom

Nt/plausible #2948

Open
wants to merge 30 commits into
base: development
Choose a base branch
from
Open

Nt/plausible #2948

wants to merge 30 commits into from

Conversation

noumantahir
Copy link
Collaborator

@noumantahir noumantahir commented Dec 26, 2024 •
edited
Loading

What does this PR?

  • recording post screen views to plausible
  • added queries to fetch stats
  • moved edit and delete actions to post screen header
  • display post views in post bottom action panel
  • added post stats display modal

Issue number

https://github.com/orgs/ecency/projects/4/views/1?pane=issue&itemId=83188613

Screenshots/Video

Screenshot 2024-12-26 at 18 46 58 Screenshot 2024-12-26 at 18 46 39 Screenshot 2024-12-26 at 13 05 09 Screenshot 2024-12-26 at 13 06 58

@noumantahir
added working plausible event recording api
835441d
@noumantahir
added api support for fetching post stats
149c70e
@noumantahir
added query hooks to interact with plausible apis
42717b7
@noumantahir
use plausible tracker hook to record post view event
fd4311a
@noumantahir
rendering page views humber in bottom action panel
b0ae35d
@noumantahir
setting url path to stats query for non cached posts
5352131
@noumantahir
introduced separate stats panel componnet
c14452c
@noumantahir
migrated profile modal to use statsPanel component
38e8958
@noumantahir
added basic post stats component for viewing plausible stats
c7e26b8
@noumantahir
using post stats component in post display view
246b659
@noumantahir
added support for fetching and caching post stats by country
462789f
@noumantahir
rendering country stats
94fe2d7
@noumantahir
updated stats panel basic margin
dcad8ab
@noumantahir
updated minutes presentation
5869c85
@noumantahir
returning sorted country data
43f0989
@noumantahir
code cleanup
ad44515
@noumantahir
lint
319d1a7
@noumantahir
lock file update
4fa05f1
@noumantahir
refactored abbreviated number method
76fb641
@noumantahir
fine tuned post botom action panel
beaca94
@noumantahir
added delete post option in post options dropdown
1b8521a
@noumantahir
added edit post options on post screen header
1e6e2fa
@noumantahir
removed delete and edit options from post action panel
9b76072
@noumantahir
lint
8b1cabf
@noumantahir
allowing dynamic dimension stats fetch and casting/conversion
7d3c0dc
@noumantahir
removed country stats item
ea5595f
@noumantahir
added device stats
0c811ac
@noumantahir
basic reveal animation for device stats
b9ea8d1
@noumantahir
added promote encouragement message
e3dba30
@noumantahir noumantahir requested a review from feruzm December 26, 2024 13:53
@noumantahir noumantahir marked this pull request as ready for review December 26, 2024 13:54
feruzm
feruzm reviewed Dec 26, 2024
View reviewed changes
yarn.lock Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@feruzm feruzm feruzm left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@noumantahir @feruzm