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.

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

Plugin Improvements #80

Open
Fenn-CS opened this issue Oct 15, 2018 · 20 comments
Open

Plugin Improvements #80

Fenn-CS opened this issue Oct 15, 2018 · 20 comments

Comments

@Fenn-CS
Copy link
Contributor

Fenn-CS commented Oct 15, 2018

Thank you awesome developer of this plugin! I choosed your plugin over RainLab.Blog because of the news letter ability and simplicity with which the variables are made available on the page. Also I personally prefer it's interface simple and better. However advanced users may find the plugin incomplete for the following reason

  1. post.user not available? Authors of posts should be available as a key feature
  2. Things like post.next and post.prev and a pagination for posts (The pagination code on rainlab.blog seems to work on indikator.news though) as those are traditional requirements in blog iterations
  3. It's not particularly clear on how to go about email settings like changing default messages and all.
  4. Despite the simplicity of Rainlab.Blog hundred of developers have built extension plugins for it so developers are likely to choose to save some time. For example there's a Blog Views counter for RainLab.Blog which counts page views and produces a component for popular posts this is (or was not available for Indikator.News) I however created a copy of Blog Views for indikator you can find it here : https://github.com/Fenn-CS/newspageviews works exactly the same way it would work on RainLab.Blog.
  5. Since a lot of developers are building on RainLab and not Indikator I think it has to ship with a lot of tools out of box to make it more appealing.

Available to help if need be, just like newspageviews I would be building (or copying if you want) more extensions for Indikator.News but please try to fix this basic issues and thank you for the work so far!!

@gergo85
Copy link
Owner

gergo85 commented Oct 18, 2018

Thank you for your useful feedback! I will fix this issues as soon as. I'm glad you're making extensions for it.

@Fenn-CS
Copy link
Contributor Author

Fenn-CS commented Oct 19, 2018

Great. Following up :)

@Fenn-CS
Copy link
Contributor Author

Fenn-CS commented Oct 19, 2018

First plugin on Indikator.News now available in OctoberCMS store.
https://octobercms.com/plugin/fenncs-newspageviews

@gergo85
Copy link
Owner

gergo85 commented Oct 21, 2018

I added your plugin to the documentation.

@Fenn-CS
Copy link
Contributor Author

Fenn-CS commented Oct 31, 2018

@gergo85 Have you added a webhook to this repository? I am asking because after the new updates I expected to see the plugin up for updates in the plugin manager but that has not been the case.

In case there's no webhook to inform the store of updates I think one should be set and if one already exist we should check why changes are not reflected.

@gergo85
Copy link
Owner

gergo85 commented Nov 8, 2018

@Fenn-CS How can I add webhook to this repository? I've never done this before.

@Fenn-CS
Copy link
Contributor Author

Fenn-CS commented Nov 8, 2018

@gergo85 Please consult "Updating plugins automatically" section of this page

https://octobercms.com/help/site/authors

@gergo85
Copy link
Owner

gergo85 commented Nov 9, 2018

I will add some minor changes for plugin. After that I will update it.

@Fenn-CS
Copy link
Contributor Author

Fenn-CS commented Nov 9, 2018 via email

@gergo85
Copy link
Owner

gergo85 commented Nov 14, 2018

Maybe I will change the layout. But it is a WYSIWYG plugin's bug.
Also I added Author field to the form.

@Fenn-CS
Copy link
Contributor Author

Fenn-CS commented Nov 14, 2018 via email

@gergo85
Copy link
Owner

gergo85 commented Nov 30, 2018

Finally I have changed the layout a bit because of tag list field.

@Fenn-CS
Copy link
Contributor Author

Fenn-CS commented Dec 3, 2018

@gergo85 ok great! Thanks for the updates.

@Tetraneon
Copy link

Tetraneon commented Dec 7, 2018

Many thanks to your very useful update (author, prev and next, etc.).

Would it be abused to ask a "prevcat" and a "nextcat" to have the previous and then next post in the same category ?
Best regards

@Fenn-CS
Copy link
Contributor Author

Fenn-CS commented Dec 7, 2018 via email

@Tetraneon
Copy link

Hi Fenn-CS,
Of course I can do that... I would use a new post.prev() fonction ("prevcat") , to navigate between posts but in the same category.

@snipiba
Copy link

snipiba commented Jan 8, 2019

@gergo85 perfect work bro. maybe some usefull hints for next work>

  • extending for seo (keywords, description, title, og image)
  • issue at /news/statistics ErrorException: Undefined variable: top in /plugins/indikator/news/controllers/Statistics.php:84 ...

at line 83 was $top .= '<br>'; and may be $this->vars['top'] .= '<br>';

@gergo85
Copy link
Owner

gergo85 commented Jan 10, 2019

@snipiba Thank you for your remark!
I fixed the statistics issue and I added the seo options to my to-do list.

@gergo85
Copy link
Owner

gergo85 commented Mar 8, 2019

@snipiba I added SEO fields to news.

@snipiba
Copy link

snipiba commented Nov 21, 2019

@gergo85 ... maybe next time... something like author bio or profile, to create a linkable profile with author articles, biography, links to social media, avatar, etc. what u think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants