-
Notifications
You must be signed in to change notification settings - Fork 0
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
Improve the tutorial and issue sending a POST request from Botsing server #5
Comments
Hi Mohamed, It should be something like: |
Thanks @luandrea. |
Hi @mboussaa, This is an example of a .botsing file:
|
Hello @luandrea So, I updated the config according to your helpful comments: Nothing happens when creating the issue. Am I missing something? cc @MaelAudren |
Did you enabled the GitHub Botsign application that you created to access to your repository? |
Hi @mboussaa , |
I see a different URL published from ngrok from what you have configured on GitHub, can you check it? |
Because I restarted the server to see if it fixes the problem. In the screenshot it is the old one but in my config I use the new one http://20483059.ngrok.io/botsing-server |
How can I check that? |
Can you try using http://643c3eb7.ngrok.io/botsing-github-app in the Webhook URL? |
That is it. At least now, the app started to communicate with the server. Though, got
|
the log message comes from this line of code https://github.com/STAMP-project/botsing-server/blob/master/src/main/java/eu/stamp/botsing/controller/github/GitHubAppController.java#L61 |
Ok, I have updated the README.md to make it more clear. If you have any suggestion on making it better let me know. Now for that error can you try to add the botsing label to the issue? |
Hello @luandrea Do you have any idea about that issue and are you going to consider to fix that InvalidActionException? About the readme, I'm going to send a PR to make it more clear and simple to follow. For the botsing label, OK. |
Hi @mboussaa, Thank you very much for your feedback and I will wait for your PR to add also these last changes. |
I create this PR to fix the guide steps #6 |
For the exception, I don't know if you could reproduce the issue, or plan to fix it... |
Dear @mboussaa , |
Hello @cformisano
|
I have had a look at the version on the testbed (that is up-to date) and I did not see your log. Which instance are you using? Is it correctly configured? Looking at the readme file I see that it is not specified: I am going to fix it, however please, configure your instance as suggested and let me know. |
I followed all steps as in the readme file
|
sounds strange. Seems like two different version. Let's check to align: for this fix I did not change the version number, I will do it. I am defining a version 1.1.1 that I am going to push on GitHub, please use this one. |
Changing the version does not fix the issue as I am doing mvn clean package everytime after fetch/pull. |
Very strange: please, double check the name of the file: I see that you copied the name "application.propoerties" (the correct spelling is application.properties. |
that is because I wrote it quickly :D yes the file is located here: src/main/resources/application.properties |
in the deployment, is it located in the same directory of the jar file? |
@cformisano I don't understand your question. Here is where my files are located: |
@mboussaa , sorry for the late answer (I was ooo yesterday and the day before). |
same issue |
OK. Since for me it is impossible to reproduce the issue I see two possibilities:
If (as I guess) this is not possible please: I will try to understand what is happened and I will return to you with a solution. |
-java -jar botsing-server-1.1.1.jar --spring.config.location=/Users/mohamedboussaa/Documents/STAMP/botsing-server/src/main/resources/application.properties (also tested /Users/mohamedboussaa/Documents/STAMP/botsing-server/target/application.properties and also without --spring.config.location)
|
thank you very much, now it is clear.
now indicates the correct label "botsing", so this is OK. Furthermore the option
is not correct, you should use
however I think that it should not be necessary since the log is OK now (and that option is not taken into account by the app) If you want to start botsing you have to define a label "botsing" (lowercase not "Botsing") and use it. Please, let me know if it works now. |
ouf that is it. I didn't add the label. It was not mentioned in the tutorial though.
|
Dear Mohamed, |
Thank you for your reply. Here is the link for .botsing https://github.com/mboussaa/Botsing-server-test/blob/master/.botsing |
It is identical to the mine.
|
I get this:
|
It seems that the file is not present in the same folder of the pom |
You don not have to show it to me, maven complains of this :). |
Hi,
I was trying to follow the steps described in the readme file but I could not understand much how to proceed. I explain how I proceeded:
-In application.properties: I set the
githubOAuth2Token
andserver.port
-mvn clean package
-java -jar target/botsing-server-1.0.0.jar
-I escaped the
Run it under corporate proxy
part-In New GitHub App, what are the
homepage url
andwebhook URL
Could you please provide examples of application.properties and GitHub App field values that I can follow?
Thanks.
Mohamed Boussaa (AEeon)
The text was updated successfully, but these errors were encountered: