Search

Ideas that are…

Search Ideas


2225 ideas match your query.:

That means duplicate functionality; anytime I customize Devise in the future, I’ll have to remember to adjust this one method as well.

#2693​·​Dennis HackethalOP, 4 months ago​·​CriticismArchived

I could override authenticate_user! in the DiscussionsController.

#2692​·​Dennis HackethalOP, 4 months ago​·​CriticismCriticized1Archived

I could extract discussions#show into a new, separate StreamController or something like it. That controller would not use Devise.

#2691​·​Dennis HackethalOP, 4 months ago​·​CriticismArchived

Then again, I’d want to redirect users to the sign-in page (and then ideally back to where they were trying to go).

#2690​·​Dennis HackethalOP, 4 months ago​·​CriticismArchived

The thread suggests a workaround: 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.)

#2688​·​Dennis HackethalOP revised 4 months ago​·​Original #2672​·​CriticismCriticized1Archived

The thread suggests 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.)

#2686​·​Dennis HackethalOP revised 4 months ago​·​Original #2672​·​CriticismCriticized1Archived

Duplicate of #2677/#2683.

#2685​·​Dennis HackethalOP, 4 months ago​·​CriticismArchived

While ideas load, cmd +f won’t work.

#2684​·​Dennis HackethalOP, 4 months ago​·​CriticismArchived

I could render the first ~10 top-level ideas immediately and only render the rest as turbo frames off screen. By the time the user scrolls down, they should all be loaded.

#2683​·​Dennis HackethalOP, 4 months ago​·​CriticismArchived

Yes, it would be even worse than #2677 (see criticism #2681), where only top-level ideas were turbo frames.

#2682​·​Dennis HackethalOP, 4 months ago​·​Archived

Too many requests when there are enough top-level ideas.

#2681​·​Dennis HackethalOP, 4 months ago​·​CriticismArchived

I just tried this. Seemed promising at first but sometimes ideas load out of order. Looks horrible.

#2680​·​Dennis HackethalOP, 4 months ago​·​CriticismCriticized1Archived

cmd + f won’t work reliably.

#2679​·​Dennis HackethalOP, 4 months ago​·​CriticismCriticized2Archived

That means cmd + f won’t always work.

#2678​·​Dennis HackethalOP, 4 months ago​·​CriticismArchived

The top level ideas could be rendered as turbo frames of ideas#show.

#2677​·​Dennis HackethalOP, 4 months ago​·​Criticized2Archived

For large discussions, wouldn’t that flood the server with requests?

#2676​·​Dennis HackethalOP, 4 months ago​·​CriticismArchived

I could have a separate route at /ideas/:id/isolated which renders only the idea without any parents or children. And then a discussion could render a bunch of deeply nested turbo frames loading that route.

#2675​·​Dennis HackethalOP, 4 months ago​·​Criticized1Archived

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.)

#2673​·​Dennis HackethalOP revised 4 months ago​·​Original #2672​·​CriticismCriticized1Archived

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

#2672​·​Dennis HackethalOP, 4 months ago​·​CriticismCriticized1Archived

JS modules are always deferred and unusable until the page is fully loaded. As a result, comment buttons and gutters won’t work while ideas are still streaming onto the page.

#2671​·​Dennis HackethalOP, 4 months ago​·​CriticismCriticized1Archived

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

#2670​·​Dennis HackethalOP, 4 months ago​·​CriticismCriticized1Archived

Feature idea: pay people to address criticisms (either revise an idea and check off criticisms or counter-criticize).

#2669​·​Dennis HackethalOP, 4 months ago​·​CriticismCriticized1Archived

This page used to take ~3.5 seconds to load. Now it renders within 600ms :)

#2668​·​Dennis HackethalOP, 4 months ago​·​Archived

‘Veritula’ is a difficult name, people don’t know how to spell or pronounce it. They can’t easily remember it.

#2666​·​Dennis HackethalOP revised 4 months ago​·​Original #2653​·​CriticismCriticized2Archived

‘Renaissance’ isn’t exactly easy to spell either.

#2665​·​Dennis HackethalOP, 4 months ago​·​CriticismCriticized1Archived