photogabble / isso-php Goto Github PK

It is useful to add List-Unsubscribe header (RFC8058) to e-mail with conversation notification. This header will allow user to unsubscribe from e-mail notification via one-click in her e-mail application.

This test fails because url validation for #36 isn't yet implemented. Once that is done both this issue and #36 can be closed.

Using vlucas/phpdotenv is probably the best bet here.

Refactor CommentTest::testGetLimited so that it pre fills the db with 20 comments and only makes one request.

This is a sibling of #37

Adding support WebMention (Tests) would be nice.

• comments.new:new-thread
• comments.new:before-save
• comments.new:guard
• comments.new:after-save
• comments.new:finish

This relates to https://github.com/posativ/isso/blob/master/isso/db/comments.py#L176.

This task involves porting the SQL query that the Python version builds into a Doctrine query object.

Use UUID for identifying users rather than ip (due to ip being an unreliable identifier).

The java-script can set a UUID and store it locally in the browser so that the backend can identify which comments belong to the current visitor.

Throughout the codebase I have been returning new TextResponse instances with the error messages contained. When what should be happening is an exception thrown with the error and its code and then the exception handler deciding if the response should be text, json, etc based upon request headers.

This will allow debug mode to output a stack trace and bits if needed.

Add a configurable time limit from the first comment, after which all replies are disabled.

Using encrypted one-time-use session links we can allow the admin to accept/decline comments in the moderation queue.

Refactor CommentTest::testCreateAndGetMultiple so that it refills the database with 20 comments for the /path/ uri and then only makes one request.

Hi,

I've just saw your project and really want to try it for my blog.
However as I'm not php-savvy I do not know how to run it.

Could you help me with setting up isso-php ? Thanks.

PS: Php installed, Nginx and Sqlite3 too.

See https://github.com/posativ/isso/blob/master/isso/db/__init__.py#L98

This is a hard coded value in isso and I am not sure if the front end js supports greater nesting depths. Should this be enhanced?

/*
 * Maximum number of levels of replies that can be made.
 */
'max-nesting' => 1,

Add a basic validator for the query params that can be used by other methods to validate their input.