search

ServerPress / sync-genesis

Sync for Genesis Settings
1 stars 1 forks source link

Address TODO issues #1

Open davejesch opened 8 years ago

davejesch commented 8 years ago

@cleverness I reviewed the code for this plugin and made several comments in the code. These are all marked with a // TODO: comment. Please find and address these in both the .php and .js files.

cleverness commented 8 years ago

These were fixed in the following commits: https://github.com/ServerPress/sync-genesis/commit/97b5f9b3d5e0bd3a34a75db455a29bc6ea66e42c https://github.com/ServerPress/sync-genesis/commit/88dac547e09ee8717aea6affea963d7f18ab58f0 https://github.com/ServerPress/sync-genesis/commit/14e6654f0fbc0e7329482fd2a597bc1d2b45c05c https://github.com/ServerPress/sync-genesis/commit/4d35a7e6f9d4128d0f6a81e13abef07dc74d33ab https://github.com/ServerPress/sync-genesis/commit/98a708be27c4bca4fe66585135d5ea9c18e93023 https://github.com/ServerPress/sync-genesis/commit/6325f2d8982b947e6393c89509dea1588b013649

davejesch commented 8 years ago

@jennifercloake All items appear to be addressed. We can open this up for further testing.