-
Notifications
You must be signed in to change notification settings - Fork 8
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
Some things to review #23
Comments
|
|
How about user signup and login? To edit his own events |
"No users" This thing was a very early requirement in the project hence there are no plans for implement this. But, OTOH this is a thing that I have been thinking for long time and the conclusion reached is: a unique key delivered to the speaker's email could solve the issue because this key should be submitted in order to edit the event. Any comments/thoughts about that? |
Who the hell wrote that requirement? A gardener? A system administrator? 😆 A random generated token (unique key) would be nice, the token should be associated to an email. But this is a user and a password for it, isn't it? |
Add: Notifications when someone writes a comment on yours event. |
Add: Markdown or any markup language for Description and/or comments. Check this out. |
Yes, maybe a sysadmin... To be precise was @queltosh , you have to argue this issue with him :). You have to keep in mind that the app is used a couple times per year more or less, and store users in the DB would be useless and maybe risky. The token is a definitely easier, for the speaker and for us, because we don't have a profile itself with a username, password, other info, etc just a token associated to your email, but of course I know that is nearly the same. And of course the random key is delivered after submit an event or after click the accept link in the email. |
Store users on a database is useless and risky? @guluc3m/owners are storing mail lists passwords on plain text and nobody gives a fuck... But this is not the topic now. I am agree with token based authentication and it would be nice to use the same token for the API to make authenticated requests. |
I'm not too keen on the idea of having users (apart from admins or assistants), just saying. Also, there is no implementation for performing authenticated requests in place. |
Show language selector in header1.0.1b or 1.5.0b2.0.0b > 2.1.0Convert old database to the new structure to show the past editions.Substitute ActiveAdmin with RailsAdmin or better.Cleanup locales and include some forgoten strings.(DUPLICATED)Add statistics page (see chartist-js)The text was updated successfully, but these errors were encountered: