Search

Ideas that are…

Search Ideas


2221 ideas match your query.:

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

The following commits should address this:

  • 3af3966 Clarify in title that someone revised an idea (rather than originated idea)

    The HTML title now says ‘Idea x revised by…’

  • 6c70cea Underneath idea, indicate that someone revised an idea (rather than submitted it)

    It says ‘Dennis Hackethal, 1 day ago’ for new ideas, ‘Dennis Hackethal revised 1 day ago’ for revisions

  • d20d386 Explain that users can revise each others’ ideas

    As part of the alert on the revision page, when the user is about to revise someone else’s idea.

  • c5748e3 Turn ‘revise’ link into ‘revise their idea’ when it’s someone else’s idea

    Underneath each idea.

  • e0fbd41 List user under each revision in version history

    So that each version is clearly attributed to the corresponding user.

  • 06d3241 List contributors at top of version history

    Comma-separated list to see all contributors at a glance. Eg see here

#2663​·​Dennis HackethalOP revised 4 months ago​·​Original #439​·​CriticismArchived

Changing the view logic so that the controller can stream each deeply nested idea separately sounds non-trivial. It’s not clear to me how to do that currently. It may be possible someday if I adjust Hiccdown to allow the use of enumerators or something like that.

#2661​·​Dennis HackethalOP revised 4 months ago​·​Original #2658​·​CriticismArchived

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

#2659​·​Dennis HackethalOP revised 4 months ago​·​Original #2646​·​ Battle testedArchived

Changing the view logic so that the controller can stream each deeply nested idea separately sounds non-trivial. It’s not clear to me how to do that currently, It may be possible someday if I adjust Hiccdown to allow the use of enumerators or something like that.

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