Skip to content

fix news list page, news list content and pagination links#68

Merged
norbertlaposa merged 3 commits into
masterfrom
news_list_fix
May 15, 2024
Merged

fix news list page, news list content and pagination links#68
norbertlaposa merged 3 commits into
masterfrom
news_list_fix

Conversation

@Kuchtin
Copy link
Copy Markdown
Contributor

@Kuchtin Kuchtin commented May 10, 2024

Bugs connected to this:

  • Adding new article was not succesful because of missing node controller
  • Missing paginations
  • After list refresh, pagination redirected to onyx request instead of page
  • Adding new article in News List used as content did not have blog post ID and needed item.parent ID instead

Copy link
Copy Markdown
Member

@norbertlaposa norbertlaposa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The fix should be in templates files only.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to create this file.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

supply parameter in templates/bo/component/news_add.html and templates/bo/component/news_list.html via refreshNewsList('{_SERVER.HTTP_REFERRER}');

@norbertlaposa norbertlaposa merged commit 3b0fafd into master May 15, 2024
@norbertlaposa norbertlaposa deleted the news_list_fix branch November 22, 2024 09:35
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

Successfully merging this pull request may close these issues.

2 participants