I’m updating bbpress.rb to have it deal with attachments (now it inserts a link to the attachment on the original site) and create permalinks for the old URLs. The old format is `/forums/topic/slug’. For anyone who knows how to write an Nginx redirect, it’s pretty simple to redirect that to /t/slug…

There is an option already. [immagine] Redirect old forum URLs to new Discourse URLs using permalinks packageMigrating to Discourse Redirecting Old Forum URLs to New Discourse URLs using permalinks If you’ve moved from other forum software to Discourse using o…

[immagine] vinothkannans: Are you looking for something else? No, that’s what I’m doing, though rather than having to enter the rails console and find a way to do a Permalink.create for every URL, I’m doing that automatically in the importer. Normal People should not be subjected to the Rails…

[immagine] pfaffman: better to have Discourse do the redirects than to get Nginx to do it, as adding that redirect will cause Nginx to have to process it on every url request. Actually, doing it on Nginx should be way faster than doing in Discourse. The problem is that Discourse is way easier…

Thanks, @falco . I’ll take that as a “no.” There’s no reason not to have the importer add a redirect for every topic, and no reason to make it easy to turn off that functionality (e.g., by setting an environment variable).

in phpbb3 importer script they already using it script/import_scripts/phpbb3/importers/permalink_importer.rb main add_normalization(normalizations, TOPIC_LINK_NORMALIZATION) if @settings.create_topic_links SiteSetting.permalink_normaliz…

Maybe having an easy turn off is a good thing. If I were doing a special custom import, I would prefer the nginx redirect route. But having the code in a reusable script that doesn’t make me do it every time is very valuable.

[immagine] Falco: One to turn it all off (default on) looks good for me. Done. But, if it’s you that we’re talking about, you can just put a # in the front of the create_permalinks call in execute. :slight_smile:

One to turn it all off (default on) looks good for me.

https://github.com/discourse/discourse/pull/4501

[immagine] pfaffman: Is there any reason not to create a permalink for every topic Nope, I can not think of any performance reasons to avoid permalink creation, it does an indexed lookup which is very cheap.

Permalink overhead

Sviluppatore

vinothkannans (Vinoth Kannan) 14 Ottobre 2016, 5:13pm 2

There is an option already.

Are you looking for something else?

Argomento		Risposte	Visualizzazioni
Non-permalink normalizations Development	11	1352	Maggio 11, 2017
Redirect old forum URLs to new Discourse URLs using permalinks Migrating to Discourse how-to	10	29623	Settembre 13, 2024
Custom permalinks Support	5	5984	Marzo 7, 2020
Redirecting the old forum urls to the discourse Support unsupported-install	18	1405	Settembre 11, 2020
Redirecting Problems Support	10	1086	Maggio 11, 2019

Permalink overhead

Argomenti correlati