Search

Ideas that are…

Search Ideas


2103 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 25 days ago·Original #2672·CriticismCriticized1

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 25 days ago·Original #2672·CriticismCriticized1

Duplicate of #2677/#2683.

#2685·Dennis HackethalOP, 25 days ago·Criticism

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

#2684·Dennis HackethalOP, 25 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, 25 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, 25 days ago

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

#2681·Dennis HackethalOP, 25 days ago·Criticism

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

#2680·Dennis HackethalOP, 25 days ago·CriticismCriticized1

cmd + f won’t work reliably.

#2679·Dennis HackethalOP, 25 days ago·CriticismCriticized2

That means cmd + f won’t always work.

#2678·Dennis HackethalOP, 25 days ago·Criticism

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

#2677·Dennis HackethalOP, 25 days ago·Criticized2

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

#2676·Dennis HackethalOP, 25 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, 25 days ago·Criticized1

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 25 days ago·Original #2672·CriticismCriticized1

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

#2672·Dennis HackethalOP, 25 days ago·CriticismCriticized1

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, 25 days ago·CriticismCriticized1

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

#2670·Dennis HackethalOP, 25 days ago·CriticismCriticized1

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

#2669·Dennis HackethalOP, 25 days ago·Criticism

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

#2668·Dennis HackethalOP, 25 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 25 days ago·Original #2653·Criticism

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

#2665·Dennis HackethalOP, 25 days ago·Criticism

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 25 days ago·Original #439·Criticism

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 25 days ago·Original #2658·Criticism

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

#2659·Dennis HackethalOP revised 25 days ago·Original #2646· Battle tested

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, 25 days ago·CriticismCriticized1