#7858 /categories URLs needs to use unique ids

v1.5.0
closed
None
General
nobody
2016-06-13
2015-03-20
No

Trove category shortnames are not always unique. E.g. there is an "education" under "Topic" and also one under "Audience :: by Industry or Sector". So clicking on one to edit (and also submitting forms to add/edit I think) will pick one arbitrarily. We should use a unique identifier such as the "trove_cat_id" field.

Discussion

  • Dave Brondsema

    Dave Brondsema - 2015-07-21

    This also occurs at /auth/user_info/skills/ urls.

    And another duplicate example is 'python' (programming language, and a license)

     
  • Dave Brondsema

    Dave Brondsema - 2016-06-02
    • status: open --> review
    • assigned_to: Dave Brondsema
     
  • Dave Brondsema

    Dave Brondsema - 2016-06-02

    Fix in branch db/7858

    To test, go to /categories URL and make sure going to one of the duplicated names keeps you in the correct distinct part of the hierarchy. Make sure you can still add and remove categories at any level. Also follow the link in the page to go over to your skills, and make sure you can add at any level there too (the skills page UI is confusing, fixing that is a job for another day)

     
  • Dave Brondsema

    Dave Brondsema - 2016-06-13
    • status: review --> closed
     
  • Dave Brondsema

    Dave Brondsema - 2016-08-22
    • Milestone: unreleased --> v1.5.0
     

Log in to post a comment.