Veritula – Meta

Showing only those parts of the discussion which lead to #1769 and its comments.

See full discussion instead
  Log in or sign up to participate in this discussion.
With an account, you can revise, criticize, and comment on ideas.

Discussions can branch out indefinitely. Zoom out for the bird’s-eye view.
Dennis Hackethal’s avatar
Dennis HackethalOP revised 13 days ago·#1775· Collapse
2nd of 2 versions leading to #1769

Include (preview of) content in idea URLs: '/ideas/123-first-30-or-so-chars-of-idea-here'.

Criticism
Dennis Hackethal’s avatar

That would mean fetching an idea to compute the path for each hashtag. Overhead?

Criticism of #1775Criticized2oustanding criticisms
Dennis Hackethal’s avatar
Dennis HackethalOP revised 13 days ago·#1771· Collapse
2nd of 2 versions

Fetching the idea is not necessary if the feature is backwards compatible. Can still just use /ideas/123.

Criticism of #1769Criticized1oustanding criticism
Dennis Hackethal’s avatar

As noted in #1777, fetching the idea actually helps. Well worth the overhead.

Criticism of #1771
Dennis Hackethal’s avatar

Seems like minor overhead. It’s not like there are tons of user-generated hashtags everywhere.

Criticism of #1769
Dennis Hackethal’s avatar

This actually helps to prevent rendering links with IDs that don’t point to any existing idea.

Criticism of #1769