Address fields configuration page is no longer visible in the menu [#2530996]

uc_store.routing.yml defines a form:

uc_store.address_fields:
  path: '/admin/store/settings/countries/fields'
  defaults:
    _form: '\Drupal\uc_store\Form\AddressFieldsForm'
    _title: 'Address fields'
  requirements:
    _permission: 'administer store'

Since uc_country took over the country settings pages, this form is no longer visible in the menu, and gives an error if you visit the path directly.

Comments

Comment #1

10 July 2015 at 21:37

longwave committed 4aecd57 on 8.x-4.x

Issue #2530996: Address fields configuration page is no longer visible...

Comment #2

longwave

he/him

English

CreditAttribution: longwave commented 10 July 2015 at 21:39

Fixed in the above commit, though the menu and breadcrumbs are still a bit messed up because the country pages are under /admin/store/config and everything else is under /admin/store/settings.

Should we move everything to /admin/store/config? (I guess to match /admin/config)

Comment #3

TR CreditAttribution: TR commented 12 July 2015 at 20:58

I was moving things to /admin/store/config gradually, as I overhauled them. That's where I think everything should be eventually, like with core. That process allows me to keep the new code (at config) and the old code (at settings) running at the same time, making it easier to do incremental changes without breaking all of Ubercart. So country got moved to config, but a lot of others haven't been moved yet. uc_tax is currently split, for example, with the TaxRate entities being accessed at /admin/store/config and the DB table rates being accessed at /admin/store/setttings.