- **status**: review --> in-progress
- **Reviewer**: Dave Brondsema
- **Comment**:

Good usage of a form with schema validation, but not using EasyWidgets :)

When creating a new notification:

* "active" could be a checkbox, and should default to checked
* impressions should default to more than 0, like 10
* User Role should show some choices, perhaps with placeholder text.  "e.g. 
Admin, Developer"

When editing a notification, impressions just shows 0.  (Typo with 
`form_value['imressions']`)

The "Site Notifications" section of `installation.rst` should be updated 
accordingly.



---

** [tickets:#8024] Site notification admin interface**

**Status:** in-progress
**Milestone:** unreleased
**Labels:** notifications sf-4 sf-current 42cc 
**Created:** Thu Nov 19, 2015 10:51 PM UTC by Dave Brondsema
**Last Updated:** Wed Dec 23, 2015 12:45 PM UTC
**Owner:** Igor Bondarenko


The site admin pages should have a way to manage site notifications.  Also, 
make sure to update the Site Notification part of the docs.


---

Sent from forge-allura.apache.org because [email protected] is subscribed 
to https://forge-allura.apache.org/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://forge-allura.apache.org/p/allura/admin/tickets/options.  Or, if this is 
a mailing list, you can unsubscribe from the mailing list.

Reply via email to