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

Update the mapping for section_names field to reflect primary category #338

Closed
1 task done
vjayaseelan90 opened this issue Oct 19, 2022 · 1 comment · Fixed by #350
Closed
1 task done

Update the mapping for section_names field to reflect primary category #338

vjayaseelan90 opened this issue Oct 19, 2022 · 1 comment · Fixed by #350
Assignees
Milestone

Comments

@vjayaseelan90
Copy link

Is your enhancement related to a problem? Please describe.

Currently both the section_names and all_sections field displays the same information. The reason why section_names is displaying all section information is because of a work around we had to do when all_sections field was not available. Since all_section is available right now, we can go ahead and update the mapping for section_names field to only reflect primary category.

Designs

No response

Describe alternatives you've considered

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@jeffpaul
Copy link
Contributor

@Sidsector9 as you're able next week, mind making this update to change the mapping in https://github.com/globeandmail/sophi-for-wordpress/blob/908c543003133902f695a164db56936f45c7f05a/includes/functions/content-sync.php#L275 so that sectionNames only contains the Primary Category when multiple categories are set? If only one Category is set, please send that. If no Category is set, then the field will be empty.

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

Successfully merging a pull request may close this issue.

3 participants