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

"name" option of apostrophe-events-pages is hardcoded, breaking subclasses unless they are explicitly configured #41

Open
boutell opened this issue Jul 26, 2019 · 0 comments
Assignees

Comments

@boutell
Copy link
Contributor

boutell commented Jul 26, 2019

If you’re wondering why your events pages for modules that extend apostrophe-events-pages under a new name never work, this is why.

apostrophe-events-pages eventually inherits from apostrophe-custom-pages, which does this for you. But it's hardcoded as the name option of apostrophe-events-pages, breaking the magic.

If we fix it in 2.x, it’ll be a bc break for people who extended it but decided to just live with this weird fact and have only one module that extends it and put apostrophe-events-page in their apostrophe-pages configuration.

And in fact we have done that ourselves.

Which means we have to keep explicitly putting name options in our subclasses of apostrophe-events-pages until 3.0.

Dang.

@boutell boutell self-assigned this Jul 26, 2019
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

1 participant