| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
There was a problem hiding this comment.
Found one issue that needs to be addressed before this is merged.
Also, since we're duplicating a lot of code between all the modules in their respective hooks.php and load.php files to handle things like avoiding conflicts with the Performance Lab plugin and outputting generator meta tags. I think there is an opportunity to consolidate these types of things to a shared file that got included in each plugin during the build process. There are tradeoffs to consider, for sure, but something we may want to consider.
Sorry, something went wrong.
Co-authored-by: Joe McGill <801097+joemcgill@users.noreply.github.com>
Agreed. Let's open an issue to think about that further. While coming up with centralized logic for this is probably trivial, we'll likely need to spend some time considering different approaches for how we handle this central logic that still needs to be somehow copied and bundled into all of the standalone plugins. |
Sorry, something went wrong.
There was a problem hiding this comment.
Sounds good to me. 👍🏻
Sorry, something went wrong.
There was a problem hiding this comment.
Thanks @felixarntz! PR look good to me.
Sorry, something went wrong.
|
@mukeshpanchal27 @10upsimon Since there was a merge conflict and since the two changes are so closely related, I've updated this PR to include the changes in the branch from #708 as well. You can ignore those changes in here for now - let's complete and merge #708 first, and after that we can review the changes actually coming from this PR here. It should only be merged after the 2.2.0 release anyway. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
Fixes #640.
Do not merge this pull request until the two plugin repositories for these two standalone plugins have been created and approved. Likely this will only be as part of the 2.3.0 milestone.
Checklist