Veritula – Meta

Showing only those parts of the discussion which lead to #2687.

See full discussion·See most recent related ideas
  Log in or sign up to participate in this discussion.
With an account, you can revise, criticize, and comment on ideas.

Discussions can branch out indefinitely. Zoom out for the bird’s-eye view.
Dennis Hackethal’s avatar
Dennis HackethalOP revised 6 days ago·#2630
Only version leading to #2687 (2 total)

Discussions are getting slower to render as they grow. It’s a rendering issue (not a db issue).

Criticism
Dennis Hackethal’s avatar
Dennis HackethalOP revised 3 days ago·#2659
Only version leading to #2687 (2 total)

I could use ActionController::Live to stream top-level ideas to the page one by one. Instant page load.

Criticized1oustanding criticism
Dennis Hackethal’s avatar

Incompatible with Devise authentication: https://github.com/heartcombo/devise/issues/2332

Criticism of #2659
Dennis Hackethal’s avatar
Dennis HackethalOP revised 3 days ago·#2673
Only version leading to #2687 (4 total)

The thread contains a suggested solution: use authenticated do … blocks in routes.rb instead of before_action :authenticate_user! in controllers.

It’s probably a good idea to do this anyway to avoid divulging the existence of routes that unauthenticated users don’t need to know exist. (They will get a 404 instead of a 401.)

Criticism of #2670Criticized1oustanding criticism
Dennis Hackethal’s avatar

Superseded by #2686. This comment was generated automatically.

Criticism of #2673