-
Notifications
You must be signed in to change notification settings - Fork 4
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
OEL-1291: News and Event content types renamed. #92
Conversation
composer.json
Outdated
"openeuropa/oe_corporate_blocks": "^4.4", | ||
"openeuropa/oe_multilingual": "^1.9", | ||
"openeuropa/oe_starter_content": "1.x-dev", | ||
"openeuropa/oe_starter_content": "dev-OEL-1290", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Temporal until this is merged.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK for me, but adapt the PR once OEL-1290 is merged
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to rename also the modules themselves.
Let's call oe_whitelabel_starter_news, oe_whitelabel_starter_event.
We need to rename the date format inside the news module.
I've found a problem with the changes in the featured media field, I'll reopen 1290.
Content types renamed to make it work with oe_content, tests adapted.