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

A small update to enable passing extra parameters with the request #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

maimairel
Copy link

I'm integrating a slightly modified version of your tweet-js php script into Wordpress. The PHP script will then be able to fetch the API keys from the Wordpress options.

The change in the JS file I made is to accommodate Wordpress AJAX requests that needs an extra 'action' parameter.

Thanks :)

I'm integrating a slightly modified version of your tweet-js php script into Wordpress. The PHP script will then be able to fetch the API keys from the Wordpress options.

The change in the JS file I made is to accommodate Wordpress AJAX requests that needs an extra 'action' parameter.

Thanks :)
@Qodux
Copy link

Qodux commented Jun 14, 2013

Unfortunately this doesn't help me. I need to find a way to include my theme options in index.php file as StanScates suggested... This is the easiest way.

// get all theme options 
include (TEMPLATEPATH . '/framework/theme-files.php'); // this will get global variables from theme options

But the problem is that my theme_get_option function for every field (consumer key, consumer secret and so on) is not recognized.

Regards

Anyway thanks a lot for your answer and time.

@Qodux
Copy link

Qodux commented Jun 17, 2013

Well finally I found the solution...

For those who are trying to integrate Tweet.js-Mod with WP and get the 4 key’s from theme options…

http://pastebin.com/cm6bDM2k

Hope this helps and saves your itme…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants