Skip to content

Add functionality to make content url's private#716

Merged
palnabarun merged 1 commit intomasterfrom
private-slides
Aug 5, 2020
Merged

Add functionality to make content url's private#716
palnabarun merged 1 commit intomasterfrom
private-slides

Conversation

@palnabarun
Copy link
Member

The content url if set to private can only be viewed by the author or
the reviewer.

Fixes #713

@palnabarun palnabarun force-pushed the private-slides branch 4 times, most recently from f3f6d5c to 7120ad5 Compare July 31, 2020 20:58
@sayanchowdhury
Copy link
Member

@palnabarun Since you have the docker-compose files already ready. Is it possible to push them? I'll then setup the project to test.

Copy link
Member

@abhishekmishragithub abhishekmishragithub left a comment

Choose a reason for hiding this comment

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

LGTM 👍
Thanks @palnabarun 😃

@palnabarun
Copy link
Member Author

@sayanchowdhury vide our offline discussion, you can test the changes in Python 3.7 virtual environment easily.

Copy link
Member

@sayanchowdhury sayanchowdhury left a comment

Choose a reason for hiding this comment

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

@palnabarun Sure, I did look through the code and it looks good. I haven't tested it locally. Given @abhishekmishragithub already gave a +1 go ahead and merge.

Copy link
Member

@anistark anistark left a comment

Choose a reason for hiding this comment

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

Haven't tested though but looks good.

The content url if set to private can only be viewed by the author or
the reviewer.

Fixes #713

Signed-off-by: Nabarun Pal <pal.nabarun95@gmail.com>
@palnabarun
Copy link
Member Author

Thank you everyone for reviewing! :)

@palnabarun palnabarun merged commit 3fa2076 into master Aug 5, 2020
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.

Add option to make slides private

4 participants