Dennis Hackethal’s avatar

Dennis Hackethal

@dennis.hackethal​·​Joined Jun 2024​·​Ideas
Log in or sign up to follow Dennis or post on their wall.

Founder Veritula.
Author. Software engineer. Ex Apple. Translator of The Beginning of Infinity.
dennishackethal.com

  Dennis Hackethal revised criticism #2686.

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

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

  Dennis Hackethal revised criticism #2673.

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

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

  Dennis Hackethal criticized idea #2636.

On initial page load, I could just load the first ten or so top-level ideas and their immediate children, just to reduce wait times and populate the page. Then load the rest asynchronously.

#2636​·​Dennis HackethalOP, 6 months ago

Duplicate of #2677/#2683.

  Dennis Hackethal criticized idea #2677.

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

#2677​·​Dennis HackethalOP, 6 months ago

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

  Dennis Hackethal addressed criticism #2680.

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

#2680​·​Dennis HackethalOP, 6 months ago

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.

  Dennis Hackethal commented on criticism #2676.

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

#2676​·​Dennis HackethalOP, 6 months ago

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

  Dennis Hackethal criticized idea #2677.

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

#2677​·​Dennis HackethalOP, 6 months ago

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

  Dennis Hackethal criticized idea #2677.

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

#2677​·​Dennis HackethalOP, 6 months ago

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

  Dennis Hackethal criticized idea #2659.

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

#2659​·​Dennis HackethalOP revised 6 months ago

cmd + f won’t work reliably.

  Dennis Hackethal criticized idea #2635.

I could lazy load ideas: only load the parts of the page that would be visible on the current viewport. Then load more parts as the user scrolls.

#2635​·​Dennis HackethalOP, 6 months ago

That means cmd + f won’t always work.

  Dennis Hackethal commented on criticism #2630.

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

#2630​·​Dennis HackethalOP revised 6 months ago

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

  Dennis Hackethal criticized idea #2675.

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, 6 months ago

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

  Dennis Hackethal commented on criticism #2630.

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

#2630​·​Dennis HackethalOP revised 6 months ago

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.

  Dennis Hackethal revised criticism #2672.

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

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

  Dennis Hackethal addressed criticism #2670.

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

#2670​·​Dennis HackethalOP, 6 months ago

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

  Dennis Hackethal criticized idea #2659.

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

#2659​·​Dennis HackethalOP revised 6 months ago

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.

  Dennis Hackethal criticized idea #2659.

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

#2659​·​Dennis HackethalOP revised 6 months ago

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

  Dennis Hackethal posted criticism #2669.

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

  Dennis Hackethal commented on idea #2659.

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

#2659​·​Dennis HackethalOP revised 6 months ago

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

  Dennis Hackethal revised criticism #2653.

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

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

  Dennis Hackethal criticized idea #2654.

Idea: ‘The Second Renaissance’, ‘2nd Renaissance’, ‘2R’ for short.

#2654​·​Dennis HackethalOP, 6 months ago

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

  Dennis Hackethal revised criticism #444.

The following commits should address this:

  • 3af3966 Clarify in title that someone revised an idea (rathen 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

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

  Dennis Hackethal revised criticism #2658.

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.

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.

  Dennis Hackethal revised idea #2646. The revision addresses idea #2658.

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

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

  Dennis Hackethal criticized idea #2646.

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

#2646​·​Dennis HackethalOP, 6 months ago

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.