Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

feat: add support for ajaxParams in ProcessPageList() #302

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

Open
wants to merge 1 commit into
base: dev
Choose a base branch
Loading
from

Conversation

BernhardBaumrock
Copy link
Contributor

I'm working on a basic counter module and I want to show page hits on a dedicated admin page:

image

I want to show page hits only on that page, but not on the regular page list or in navbar dropdowns.

With this PR this can easily be done by adding "ajaxParams" to the pagelist config:

    wire()->modules->get('ProcessPageList');
    $form->add([
      'name' => 'chart',
      'type' => 'markup',
      'label' => 'Pageviews',
      'icon' => 'line-chart',
      'value' => "
        <div id='chart'></div>
        <script>
        $('#chart').ProcessPageList({
          rootPageID: 1,
          showRootPage: true,

          // set custom rockcounter param
          ajaxParams: {'rockcounter': true},
        });
        </script>",
    ]);

And on the server side in the hook check for the parameter:

  protected function addPageListCounter(HookEvent $event): void
  {
    $p = $event->arguments('page');
    if ($p->template == 'admin') return;

    // if rockcounter param is not present we don't add the counter badge
    if (!wire()->input->get('rockcounter')) return;

    $event->return = $this->partial('badge') . $event->return;
  }

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.

1 participant
Morty Proxy This is a proxified and sanitized view of the page, visit original site.