-
Notifications
You must be signed in to change notification settings - Fork 2
[refactor] Forum email digest settings #262
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
base: master
Are you sure you want to change the base?
Conversation
|
Hi, Jenny! It looks like there are more places in the code where changes need to be made. I did a search across the codebase for
There are likely other files as well, for example here is one more from the plugin:
Let's talk in our meeting today to map out next steps. |
I'll move forward for figuring out the bug. I also need to look at this xml file. But I am unsure at the moment what we want to do with the /components/com_groups/admin/views/manage/tmpl/edit.php file. It's for the admin and I can remove the section to let admins change email settings in the view. I can look further into the core library to see what that file does. |
I lied. If I take out the lines that creates the section in the view completely instead of commenting it out, there is no error. |

I also added in backup files for a small handful of page sites, hope that is ok.