Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Dynamic callback URL + updated description #49

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Dynamic callback URL + updated description #49

wants to merge 2 commits into from

Conversation

vinceve
Copy link

@vinceve vinceve commented Feb 10, 2013

Hi,

I added a setter for callback URL to make it dynamic. Now it should be possible to determine at runtime the callback url. This could be used in specific places.

I also edited the README file for a more accurate description.


#app/config/config.yml
fos_twitter:
file: %kernel.root_dir%/../vendor/twitteroauth/twitteroauth/twitteroauth.php
file: %kernel.root_dir%/../vendor/kertz/twitteroauth/twitteroauth/twitteroauth.php
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simply remove this line entirely. It is not needed when the class can be autoloaded, which should be the case when installing the lmibrary through Composer (otherwise, it should be fixed in their composer.json)

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stof : #45 ;)

@bamarni
Copy link

bamarni commented Feb 11, 2013

@vinceve : I've submitted a PR a few days ago with almost the same changes, imo you should just keep the method for the callback.

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

Successfully merging this pull request may close these issues.

4 participants