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

pkp/pkp-lib#3575 Add consent statements to user registration and notifications #537

Merged
merged 5 commits into from
May 24, 2018

Conversation

NateWr
Copy link
Contributor

@NateWr NateWr commented May 18, 2018

No description provided.

@bozana
Copy link
Contributor

bozana commented May 22, 2018

Ah, I see the tests are failing here -- I can take a look -- else everything fine...

@NateWr
Copy link
Contributor Author

NateWr commented May 22, 2018

Thanks!

@bozana
Copy link
Contributor

bozana commented May 22, 2018

@NateWr, could you overtake this change: bozana/pkp-lib@b684ff6.
I am not sure if this one as well: bozana@a5ef897.

@bozana
Copy link
Contributor

bozana commented May 23, 2018

Ah, a test failed -- I'll see... -- I only tested some and then thought the others will then work too... but apparently... :-)

@bozana
Copy link
Contributor

bozana commented May 23, 2018

@NateWr, this change is also needed: bozana@577470c :-P
I hope I hope I hope that this is it :-)

@NateWr
Copy link
Contributor Author

NateWr commented May 24, 2018

Thanks @bozana! I am trying to get selenium going so I can fix these issues myself!

@bozana
Copy link
Contributor

bozana commented May 24, 2018

Tests passed 🎉
So you can merge...
Would you then also cherry-pick/migrate it all to stable-3_1_1?

@NateWr NateWr merged commit df78e35 into pkp:master May 24, 2018
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