Consolidate notices about all-in-one-seo-pack HOT 8 CLOSED

Classes should be accepted as a possible array (required) Should be notice plus any one of notice-error, notice-warning, notice-success, or notice-info. Optionally use is-dismissible to apply a closing icon.

from all-in-one-seo-pack.

Related to #25

from all-in-one-seo-pack.

check out github.com/collizo4sky/persist-admin-notices-dismissal

from all-in-one-seo-pack.

We also need to make sure we support <4.2 (which may have been the problem with our prior woocommerce notices not dismissing)

from all-in-one-seo-pack.

@michaeltorbert I noticed this issue similar to 'Add a 5 star ratings request' #61, and wanted to request assigning this.

I am aware of most of the notices by now, and took a look at the Repo for Persistant Admin Notice Dismissal. The other repo seems to apply a similar concept by working off a time-lapse, and uses a dismissal variable. Then saves it as wp_options. Not too much different than what is already in development.

One other thought that did occur to me though, is...

• Making the dismissal unique to users. So, multiple Admins users can dismiss the notice without it dismissing for others that would otherwise act on the notice.

from all-in-one-seo-pack.

@EkoJR I've assigned this two you, but we should be careful about possible conflicts. It's probably best to wait until the other one is finished before doing this.

Offhand I'd say some notice dismissals may be unique to users, but most are probably site wide.

Something a bit different but a similar subject: #998

from all-in-one-seo-pack.

Understood. I was curious if I should wait for the other branch.

from all-in-one-seo-pack.

Issue moved to awesomemotive/aioseop-pro #774 via ZenHub

from all-in-one-seo-pack.