[nodebb-plugin-calendar] Fully featured calendar plugin for NodeBB - Testers needed

frgilb

Oh oh, shame on me! Calendar is working in v1.6.0 ! I replayed my v1.5.2 backup for other reasons and forgot to update to v1.6.0 again...

@PitaJ: Please forget my previous post!

At least I can confirm, that [email protected] requires at least nodebb v1.6.0

PitaJ

New version. Please try out [email protected]
Note: this version requires NodeBB v1.6.0+

npm i nodebb-plugin-calendar@latest

Changelog:

Fix for compatibility with NodeBB v1.7.0

Sébastien

Hello,

thank you for your plugin, it is convenient

However, I have a problem, I can not validate the creation of the vent if I indicate a reminder.

Are you aware of this bug?

pl0x

Awesome work !

It is working on my v1.6.1

it work on v1.7.2 ?

ender

This post is deleted!

Michael Pfaff

Hi @PitaJ! Is this still a plugin you're working on actively? Interested in installing / testing.

PitaJ

It works just fine on v1.7.2

Michael Pfaff

@pitaj said in [nodebb-plugin-calendar] Fully featured calendar plugin for NodeBB - Testers needed:

It works just fine on v1.7.2

Great I’ll check it out!

absurdsmash

@PitaJ my community would like to see names of attendees instead of avatars, as well as a the total number of Yes/No/Maybe. How would I go about implementing this?

Edit:

So I guess in client/responses.js, change this:

  const userTemplate = user => (`
    <li class="icon pull-left">
      <a href="${config.relative_path}/user/${user.userslug}">
        ${user.picture ? `
        <img title="${user.username}" class="img-rounded user-img not-responsive"
          src="${user.picture}">
        ` : `
        <div class="user-icon user-img" style="background-color: ${user['icon:bgColor']};"
          title="${user.username}">${user['icon:text']}</div>
        `}
      </a>
    </li>
  `);

into something like this?

  const userTemplate = user => (`
    <li>
      <a href="${config.relative_path}/user/${user.userslug}">
         ${user.username}
      </a>
    </li>
  `);

of course it would be awesome to have this as an option in the acp

PitaJ

@absurdsmash would you mind opening an issue on GitHub? It helps me track feature request more easily.

Alex Galax

First of all - great plugin!

I forked the plugin. But the intallation npm i <myGhUsername>/nodebb-plugin-calendar won't create the ./build build.

I want to do some changes on the templates. Whats the best practice for this?

PitaJ

@alex-galax the best way to override templates is probably to create a new plugin with a templates directory and a plugin.json set to {}. You can override the templates that way if you copy the calendar templates over.

Alex Galax

@pitaj but I can't do it in my custom theme, because its getting build first, right?

PitaJ

@alex-galax right, your custom theme is lowest priority.

Alex Galax

@pitaj hmm, the html code for displaying an event in the topic ist done in /src/lib/template.js. So can't really overwrite this way, right?

PitaJ

@alex-galax Hmm, yeah I guess not. What exactly do you want to change? Maybe it can be accomplished with CSS?

Alex Galax

@pitaj want to change the panel layout. shure, could be done in css but its not neat
best thing for my layout would be the event as a seperate post component. and for that I need to change the plugin anyways. but i still have no idea, why the fork won't install correctly

PitaJ

@alex-galax Using CSS would be a lot better from a maintainability standpoint. What do you mean by a separate post component?

Alex Galax

@pitaj that's true. I will try how far I come with pure css.
I mean, I'd like to have it as a <li component="post" element. Why isn't that html code parsed as a tpl anyways?

PitaJ

@alex-galax having it display as a post is going to be more difficult than you might think. It will require changing now things that just the calendar plugin I think.

I used a JS template because I was using some pretty complex logic and using a normal template would have been pretty verbose. With benchpress now, though, I'll look into switching back.