These are chat archives for fossasia/open-event-server

16th
Jul 2018
Max Lorenz
@maxlorenz
Jul 16 2018 04:48
not sure. the postgres db has no attached volume so I really don't know how to achieve that
Bhavesh Anand
@bhaveshAn
Jul 16 2018 06:42
Guys Please check https://github.com/fossasia/open-event-server/pull/4737#issue-188524505
FB authentication is complete to best of my knowledge.
The response shows access_token and token_type, though the screenshot is having black marks to hide access_token a bit due to security issues. So please don't mind. :smile:
Please review.
Also, this is a priority issue and a big feature too. So it would be good to split it and take responsibilities individually. (You can follow the implementation of FB PR too) In that case this PR is ready to merge too.
Saurav Tiwary
@srv-twry
Jul 16 2018 14:44
In reference to: fossasia/open-event-server#5068
@mayank8318 Looking into this issue in detail, I found that we don't need that column for cookie policy in the user model. Users don't have to explicitly approve it. They are approving it just by using the site.
The requirement of this issue is that we should allow the admin of the system to add this information to the site. Also add a default cookie policy with the message that @mariobehling has mentioned.
Please revert your PR and simply add a cookie-policy in the settings column.
Saurav Tiwary
@srv-twry
Jul 16 2018 14:53
@bhaveshAn yeah that's a good idea to split up the work. I have created a parent issue for it: fossasia/open-event-server#5115
I have already linked the fb auth issue as it's child issue. Others can create child issues and solve them.
Till then I can implement the fb auth in the frontend.
Bhavesh Anand
@bhaveshAn
Jul 16 2018 15:04
:+1:
Saurav Tiwary
@srv-twry
Jul 16 2018 21:25
Does anyone know how I can add a test stripe key to the heroku deployment settings ?
I mean the settings endpoint of the open-event-api-dev
Pradeep Gangwar
@pradeepgangwar
Jul 16 2018 21:26
Will adding it as environment variable work?
Saurav Tiwary
@srv-twry
Jul 16 2018 21:27
No, it should in the settings.
I guess I can add it in the populate_db.py file here: https://github.com/fossasia/open-event-server/blob/development/populate_db.py
Bhavesh Anand
@bhaveshAn
Jul 16 2018 21:45
using patch /v1/settings/1 i guess @srv-twry
Saurav Tiwary
@srv-twry
Jul 16 2018 21:46
yeah but the admin email and password ?
Bhavesh Anand
@bhaveshAn
Jul 16 2018 21:47
ahh! but thats the only way using APIs
Saurav Tiwary
@srv-twry
Jul 16 2018 21:47
we must be setting the admin email and password somewhere right ?
Bhavesh Anand
@bhaveshAn
Jul 16 2018 21:50
yeah
Saurav Tiwary
@srv-twry
Jul 16 2018 21:56
Do we have a test email account for the open-event-server which I can use to register on stripe ?
Bhavesh Anand
@bhaveshAn
Jul 16 2018 21:57
not that i am aware of :thought_balloon:
Saurav Tiwary
@srv-twry
Jul 16 2018 21:59
We have test emails and accounts in other projects like the orga app and the general android app. I think we should create one for this project also.