Safety1st

Too bad all suggestions are just ignored… 

On 1/24/2017 at 9:27 PM, Adriano Faria said:

So how it would be? A new screen where you enter a name and a decription and that name should appear in all screens when you send an invitation?

No, it just a field in invitations table, for admin purposes only.

It seems there is no lang keys notifications__customnotifications_Customnotification и notifications__Customnotification for that page or they just misspelled…

The only left suggestion I found is 'campaign name':

On 7/10/2016 at 1:33 PM, Safety1st said:

Add feature to set comment for batch invitations. Then we will know from which campaign a new member joined.

On 7/16/2016 at 4:04 PM, Safety1st said:

Knowledge which campaign is worth and where a member knew about us is important for owners who invest a lot of time to their forums.

Deeply sorry for inconvenience

True. I'm a developer too and I believe that bugs & new features should be prioritized among all plugins. Especially when you really DO care about ALL of them simultaneously. Also I always try to treat differently active clients with good ideas. But that is my choice.

I understand your reaction, however, try to imagine just for one moment that implementing of my ideas makes your app better, at least they could. That is my contributions. In that topic I doesn't care how many plugins you have to maintain. I believe that found bugs must be fixed but suggestions may (or may not) be added.

First of all I clearly remember the last added 2 features.Hmm, may be I forgot to tell you about the rest... I will recheck tonight.

8 hours ago, Adriano Faria said:

Defect? No.

As you say. But the feature of your plugin could be easily bypassed using custom login handler for registration. Will you be si kind to fix this?

9 hours ago, Adriano Faria said:

Will be taken into consideration in next update

I just needed to clarify. Too bad ETA is unknown, last update was a long time ago.

@Adriano Faria I'm still waiting for implementation the requested earlier suggestion... Should I describe them again?

The next 2 serious defects:
• if an invitation code is required to pass registration it also must be requested if using custom login handler;
• invitation email must be at least on user's interface language, for now they are sent on default site language all the time. it would be nice to be able to choose language for particular invitation and after sending plugin must inform user about successful action through default black pop-up.

Give me the quote button removing!

I was suggested a new place of userbar. What if you add the ability to make this happen?


 

↓↓↓
 

@Flitterkill I've disabled Create button, also it is needed to remove the leading vertical line

@Flitterkill Thanks for so long waited update!

But what about option to remove quote button? Overquoting is just killing me, I'm ready to pay for modification!

And the new suggestion:
• make translatable 'Display the last post datetime as length of time since last post' string.

It would be nice if a notifications could be set using MCP. Then moderators could use them too.

I currently use IPS4 v4.1.16.1 with the plugin without any problems.

@Tom Irons Any news?

@Flitterkill Do you want me to remind you suggested features?

On ‎05‎.‎09‎.‎2016 at 2:02 AM, Safety1st said:

Yeah, options to remove quote button and show it on phone view would be the best.

I'm desperately need that feature. May be it's right time for a new version?

@Adriano Faria We noticed continuous errors when a user accesses to any not existed topic:

The URL of page the error occurred on was https://goldsrc.ru/topic/12345/

OutOfRangeException:  (0)
#0 /var/www/goldsrc.ru/system/Content/Content.php(81): IPS\Patterns\_ActiveRecord::load('12345')
#1 /var/www/goldsrc.ru/init.php(442) : eval()'d code(14): IPS\_Content::loadAndCheckPerms('12345')
#2 /var/www/goldsrc.ru/system/Dispatcher/Controller.php(96): IPS\forums\modules\front\forums\hook258->manage()
#3 /var/www/goldsrc.ru/system/Content/Controller.php(46): IPS\Dispatcher\_Controller->execute()
#4 /var/www/goldsrc.ru/applications/forums/modules/front/forums/topic.php(40): IPS\Content\_Controller->execute()
#5 /var/www/goldsrc.ru/system/Dispatcher/Dispatcher.php(129): IPS\forums\modules\front\forums\_topic->execute()
#6 /var/www/goldsrc.ru/index.php(13): IPS\_Dispatcher->run()
#7 {main}


BACKTRACE
#0 /var/www/goldsrc.ru/init.php(506): IPS\_Log::log('OutOfRangeExcep...', 'uncaught_except...')
#1 [internal function]: IPS\IPS::exceptionHandler(Object(OutOfRangeException))
#2 {main}

We have custom friendly URL for topics:

https://goldsrc.ru/topic/id

22 hours ago, Safety1st said:

It would be better to remove connection to PayPal at all and let members use any donate link as they wish. I mean link check modification and corresponding descriptions.

In that case link check could be:

/* Is the link not valid? */
if ( filter_var($val, FILTER_VALIDATE_URL, FILTER_FLAG_SCHEME_REQUIRED | FILTER_FLAG_HOST_REQUIRED)=== FALSE )
{
	throw new \DomainException( 'tipAuthor_notValid' );
}

The next suggestion. Move currency symbol to language string. For Russia native currency is ₽ for example.

@Flitterkill I found an incompatibility with Tip Author plugin by @Tom Irons. The details is here. Could you take a look? Access to ACP will be provided of course.

Nice idea  I hope it will be enough to receive money by contributions…
It would be better to remove connection to PayPal at all and let members use any donate link as they wish. I mean link check modification and corresponding descriptions.
 

The plugin doesn't work together with Kitchen Sink no matter what kind of plugin is higher in the list. I just didn't see the button. @Tom Irons may I share your code with KS's author to make compatible version?

Author forgot to mention fixed tabs on phone view

Circle before unread topics is custom modification described here.

The next suggestions

• ability to disable larger profile cover photo area (or even make it as small as possible) for groups which are not able to use cover photo (or even for all users which don't use it).
• add translation support to ks_topics_no_number.