Show the total number of article on a blog #158

Merged
Ilphrin merged 2 commits from master into master 6 years ago
Ilphrin commented 6 years ago (Migrated from github.com)

This is my attempt to fix #150

This is my attempt to fix #150
elegaanz (Migrated from github.com) reviewed 6 years ago
elegaanz (Migrated from github.com) left a comment

It just works, great job! The only missing details is that the strings you added in the templates are not translatable, you should add them to the po/plume.pot file, as explained here: https://github.com/Plume-org/Plume/blob/master/DEVELOPMENT.md#when-working-with-tera-templates

It just works, great job! The only missing details is that the strings you added in the templates are not translatable, you should add them to the `po/plume.pot` file, as explained here: https://github.com/Plume-org/Plume/blob/master/DEVELOPMENT.md#when-working-with-tera-templates
Ilphrin commented 6 years ago (Migrated from github.com)

Like this? :)

Like this? :)
elegaanz (Migrated from github.com) approved these changes 6 years ago
elegaanz (Migrated from github.com) left a comment

Like this. 👍

Like this. :+1:
Ilphrin commented 6 years ago (Migrated from github.com)

Yay! \o/

Yay! \o/

Reviewers

The pull request has been merged as 24f2030881.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b master master
git pull origin master

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff master
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#158
Loading…
There is no content yet.