Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not All Post Prominence Terms Appear in Metabox on Add/Edit Post Screen #956

Closed
0aveRyan opened this issue Nov 4, 2015 · 6 comments
Closed
Assignees
Milestone

Comments

@0aveRyan
Copy link

0aveRyan commented Nov 4, 2015

Discovered during C-HIT migration from 0.3 to 0.5.2:
C-HIT's custom prominence term disappeared from the metabox (add metabox function here)

Checked against a few other member sites including NPQ, Oklahoma Watch and adding terms in vagrant to a site not using any Custom Terms. NPQ's custom term showed, however Oklahoma Watch's and ones added in vagrant did not. (None of the terms are added programmatically for those themes)

These terms all contained posts and were accessible via the Quick Edit bulk edit screen, but wouldn't show up in the single post edit view.

@0aveRyan
Copy link
Author

0aveRyan commented Nov 9, 2015

Another situation where prominence terms aren't being completely registered properly... Cornell Daily Sun. Added a handful of prominence terms to handle their homepage and while not all terms are displaying on the Add / Edit page it's the slugs shown in the Post Prominence list that have me concerned, as something is interfering with their registration.

screen shot 2015-11-09 at 4 03 57 pm

screen shot 2015-11-09 at 4 03 45 pm

screen shot 2015-11-09 at 4 03 39 pm

@benlk
Copy link
Collaborator

benlk commented Nov 10, 2015

Is there another place where the options need to be added maybe?

@aschweigert aschweigert added this to the hotfix milestone Nov 30, 2015
@aschweigert aschweigert assigned rnagle and unassigned 0aveRyan Dec 17, 2015
@aschweigert aschweigert assigned benlk and unassigned rnagle Apr 12, 2016
@aschweigert
Copy link

Do we know if this is still an issue?

@benlk
Copy link
Collaborator

benlk commented Apr 12, 2016

I do not know.

@rclations
Copy link

These terms all contained posts and were accessible via the Quick Edit bulk edit screen, but wouldn't show up in the single post edit view.

The above pull request should solve this issue.

it's the slugs shown in the Post Prominence list that have me concerned, as something is interfering with their registration.

This part probably needs further investigation & testing to determine the cause and if it's still an issue.

@aschweigert
Copy link

as far as i can tell this is resolved with that latest PR, we'll keep an eye on it but closing for now. thanks @rclations!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants