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 27 days ago·#1775
Show idea #1763Show idea #17632nd of 2 versions leading to #1769 (2 total)

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

Criticism
Dennis Hackethal’s avatar
Dennis HackethalOP, 28 days ago·#1769

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 28 days ago·#1771

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
Dennis HackethalOP, 27 days ago·#1778

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

Criticism of #1771
Dennis Hackethal’s avatar
#1778
Dennis Hackethal’s avatar
#1771
Dennis Hackethal’s avatar
Dennis HackethalOP, 28 days ago·#1773

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

Criticism of #1769
Dennis Hackethal’s avatar
#1773
Dennis Hackethal’s avatar
Dennis HackethalOP, 27 days ago·#1777

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

Criticism of #1769
Dennis Hackethal’s avatar
#1777
Dennis Hackethal’s avatar
#1769
Dennis Hackethal’s avatar
#1775