acquia / df

Demo Framework - mirrored at https://git.drupal.org/project/df.git
https://www.drupal.org/project/df
18 stars 19 forks source link

URL alias patterns need adjustment #215

Closed rlnorthcutt closed 3 years ago

rlnorthcutt commented 4 years ago

On a fresh install of DF, I have enabled the DF tools content types. I checked the URL patterns, and we need to adjust a few things

1) Without enabling translations, I get english, spanish, and french translation path patterns... but they don't know about languages yet. These should likely be optional config for the df_tools_translations module, right?

2) I think we should just make "english" the default, and no need to label it as such... just "Article" is fine for the default.

3) Suggestions - We should change the name of the translations to put the language last. This makes it a bit easier to organize, I think. All articles will be together.

4) The location path is referencing the "field_location_type", but that doesn't exist. Its just "field_type"

5) Suggestion - all content types should potentially have the "type" field in the path. Then we have a cleaner default structure OOTB.

6) Suggestion - we should make a "content type" taxonomy path that matches the language version. This way, we can have a logical structure with views for all paths (views can be in another ticket) Example

url-alias-defaults
rlnorthcutt commented 4 years ago

Alternatively, we could remove the language variations of patterns to keep it simple. This will get way more complex when we have Japanese, German, etc.

kevinfunk commented 4 years ago

Article /[node:content-type]/[node:field_type]/[node:created:custom:Y]/[node:created:custom:m]/[node:created:custom:d]/[node:title]

Basic page /[node:field_type]/[node:title]

Event /[node:content-type]/[node:field_type]/[node:created:custom:Y]/[node:created:custom:m]/[node:created:custom:d]/[node:title]

Landing page [node:title]

Location /[node:content-type]/[node:field_type]/[node:title]

Person /[node:content-type]/[node:field_type]/[node:title]

@rlnorthcutt do you have any changes or can we close this?

rlnorthcutt commented 4 years ago

A few minor changes: 1) For the label, lets prefix the entity type. This keeps things grouped logically: