Skip to content
This repository has been archived by the owner on Feb 17, 2023. It is now read-only.

#error to stop them building it before updating the OAuth details #25

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

Conversation

palaniraja
Copy link

No point in building the app without a valid Oauth info.

No point in building the app without a valid Oauth info.
@jhersh
Copy link
Contributor

jhersh commented Apr 29, 2013

Thanks @palaniraja. Have you seen #24 from @kohsuke? I think his pull request is a bit more flexible and automation-friendly.

Of course I can't merge either of these pulls as I'm no longer at Salesforce, but thanks for contributing :)

@palaniraja
Copy link
Author

@jhersh Sorry I didn't notice. I was hunting for the source file to change the oauth info and a link to create oauth token. Building app without any necessary input was not helpful.

I agree with you, @kohsuke solution is better as long as it throw file not found for OAuthKeys.h at build time.

Thank you.

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

Successfully merging this pull request may close these issues.

2 participants