Ajeet kumar

Results 24 comments of Ajeet kumar

Suggestion: better way to remove argument from url on https://github.com/ahmedkaludi/accelerated-mobile-pages/commit/3c2a9b7380ff3aea47a29c45a8fc07655cf12e71#diff-ab576767b896423d184ddafe2ede2310R86 & https://github.com/ahmedkaludi/accelerated-mobile-pages/commit/3c2a9b7380ff3aea47a29c45a8fc07655cf12e71#diff-ab576767b896423d184ddafe2ede2310R87 we can use remove_query_arg

can we use `adsforwp_output_functions` function exists /Class exits in place if `is_plugin_active('ads-for-wp/ads-for-wp.php') `... its written multiple times in single file `if(!is_plugin_active( 'ads-for-wp/ads-for-wp.php' )){` make it single, may be its improve...

add Escaping https://github.com/ahmedkaludi/accelerated-mobile-pages/commit/0584859b6eba5fee975e292b8a81100d8ecaee83#diff-8aecbbbcd33504681166e87f9a64758fR64

1. Add nonce verification in all wp_ajax 2. check user capability to save option before replacing debug mode 3. Show admin notice only for Administrator

This line should be inside the `if condition` `if(!empty($api_key)){` https://github.com/ahmedkaludi/accelerated-mobile-pages/commit/a8e1e4e58c14c096a93def3355cc4e1bfd6b7d83#diff-b27b16759fae0fb0d94cc6e00107c8efR309-R312

use esc_attr in place of esc_html because $class contain attribute values https://github.com/ahmedkaludi/accelerated-mobile-pages/commit/49e60b181527e896f7d90044932dcf627b8eb8e3#diff-04b38a94ddf918855eed7a68ecaa512dR1253

Update the amp_loop() documentation also

Use `nonce` in to verify the request in wp_ajax

- [x] At `Rollback to Previous Version`, add please wait or some loader After Enable it. because at the first time it took time to respond - [ ] In...

https://github.com/ahmedkaludi/accelerated-mobile-pages/commit/90c9dde0187decdd6e2d0371e82e912be8ad60ec#diff-6aa5512f7dd8d5ce60fc97b0f340912cR2905 here need page ID but passed wrong id (recent post id from global value) https://github.com/ahmedkaludi/accelerated-mobile-pages/commit/90c9dde0187decdd6e2d0371e82e912be8ad60ec#diff-6aa5512f7dd8d5ce60fc97b0f340912cR2907 why `get_the_ID()` used if we need to update on all pages https://github.com/ahmedkaludi/accelerated-mobile-pages/commit/90c9dde0187decdd6e2d0371e82e912be8ad60ec#diff-6aa5512f7dd8d5ce60fc97b0f340912cR2912 here $post->post_type...