search

opentechinstitute / luci-commotion

Commotion configuration pages for the LuCI web interface
GNU General Public License v3.0
11 stars 17 forks source link

[CLOSED] Non admin portal #375

Closed oti-tech closed 10 years ago

oti-tech commented 10 years ago

Issue by areynold Friday Dec 27, 2013 at 17:39 GMT Originally opened as https://github.com/opentechinstitute/luci-commotion-apps/pull/57

Should keep applications with multiple categories from displaying more than once in non-admin view.

To test:

  1. Allow non-admin apps
  2. Add a few applications with multiple categories (e.g., fun & community)
  3. Check non-admin apps display. All apps should be displayed, and each app should be displayed only once.

areynold included the following code: https://github.com/opentechinstitute/luci-commotion-apps/pull/57/commits

oti-tech commented 10 years ago

Comment by dismantl Friday Dec 27, 2013 at 18:38 GMT

When I visit the non-admin app portal, it only shows one app, when there is more than one on the node.

oti-tech commented 10 years ago

Comment by dismantl Friday Dec 27, 2013 at 18:42 GMT

rather, I should say that it doesn't display apps that don't have any categories

oti-tech commented 10 years ago

Comment by dismantl Friday Dec 27, 2013 at 18:46 GMT

just pushed a fix for that.