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

Added example which prints all learn events #36

Merged
merged 1 commit into from
Jun 2, 2016

Conversation

miloserdow
Copy link
Contributor

Now more meaningful example ;)

@mehanig
Copy link
Contributor

mehanig commented Jun 2, 2016

Yes, much more, and also good idea!

Print formatting is a little bit weird, but as example it's ok.
Thank you.
screen shot 2016-06-02 at 1 29 35 pm

@mehanig mehanig merged commit e3ac046 into StepicOrg:master Jun 2, 2016
@vyahhi
Copy link
Contributor

vyahhi commented Jun 2, 2016

Oops, I thought we have removed events from API.
cc @andrvb ;)

@mehanig
Copy link
Contributor

mehanig commented Jun 2, 2016

@vyahhi I've discussed with @psviderski yesterday about this endpoint, for us it's also unclear why it's open for everyone.
Might be, some pages at front-end using it, but i'm not sure if it's good idea.

@vyahhi
Copy link
Contributor

vyahhi commented Jun 2, 2016

For https://stepic.org/notifications/events, but we were going to remove it. AFAIK it's staff only now.

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.

3 participants