Search

Ideas that are…

Search Ideas


1907 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, 8 days ago·Criticism

I could override authenticate_user! in the DiscussionsController.

#2692·Dennis HackethalOP, 8 days ago·CriticismCriticized1oustanding criticism

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

#2691·Dennis HackethalOP, 8 days ago·Criticism

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, 8 days ago·Criticism

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 8 days ago·Original #2672·CriticismCriticized1oustanding criticism

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 8 days ago·Original #2672·CriticismCriticized1oustanding criticism

Duplicate of #2677/#2683.

#2685·Dennis HackethalOP, 8 days ago·Criticism

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

#2684·Dennis HackethalOP, 8 days ago·Criticism

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, 8 days ago·Criticism

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

#2682·Dennis HackethalOP, 8 days ago

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

#2681·Dennis HackethalOP, 8 days ago·Criticism

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

#2680·Dennis HackethalOP, 8 days ago·CriticismCriticized1oustanding criticism

cmd + f won’t work reliably.

#2679·Dennis HackethalOP, 8 days ago·CriticismCriticized2oustanding criticisms

That means cmd + f won’t always work.

#2678·Dennis HackethalOP, 8 days ago·Criticism

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

#2677·Dennis HackethalOP, 8 days ago·Criticized2oustanding criticisms

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

#2676·Dennis HackethalOP, 8 days ago·Criticism

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, 8 days ago·Criticized1oustanding criticism

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 8 days ago·Original #2672·CriticismCriticized1oustanding criticism

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

#2672·Dennis HackethalOP, 8 days ago·CriticismCriticized1oustanding criticism

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, 8 days ago·CriticismCriticized1oustanding criticism

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

#2670·Dennis HackethalOP, 8 days ago·CriticismCriticized1oustanding criticism

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

#2669·Dennis HackethalOP, 8 days ago·Criticism

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

#2668·Dennis HackethalOP, 8 days ago

‘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 8 days ago·Original #2653·Criticism

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

#2665·Dennis HackethalOP, 8 days ago·Criticism