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

Add employment application functionality #183

Closed
stellanl opened this issue Dec 27, 2015 · 4 comments
Closed

Add employment application functionality #183

stellanl opened this issue Dec 27, 2015 · 4 comments

Comments

@stellanl
Copy link
Contributor

This lets an Up user apply to be connected to an organisation without using the Web interface, as discussed in #159

stellanl added a commit that referenced this issue Dec 27, 2015
storage for more columns and fetching of all organisations.
@stellanl stellanl self-assigned this Apr 1, 2016
@stellanl stellanl added this to the v1.6 Release - Ferdinand milestone Apr 1, 2016
@stellanl
Copy link
Contributor Author

stellanl commented Apr 1, 2016

Just found out that the /rest/v1/user//request_organisation/ entry point requires Django login as well as API key. akvo/akvo-rsr#2074

stellanl added a commit that referenced this issue Apr 14, 2016
all fields and completion for organisations and countries. Begin
preparing for localization.
stellanl added a commit that referenced this issue May 10, 2016
popups for sent data. Keep employments for different users separate.
@stellanl
Copy link
Contributor Author

One way to improve the UX might be to load organisations and countries as part of the login, if this is the first login. It will take tens of seconds, but it's a one-time occurrence, and the user will have no expectations. The db will then have all necessary info for a connection application; user will not need to do a refresh on that screen.

@stellanl
Copy link
Contributor Author

Turns out the country must exist in
https://github.com/akvo/akvo-rsr/blob/develop/akvo/codelists/store/codelists_v202.py
and the comparison is case-sensitive, so we cannot use the country code obtained from /rest/v1/country/ without upper-casing it first.

@stellanl
Copy link
Contributor Author

stellanl commented Oct 13, 2016

Application submissions now work, but the approval Email sent to the user could be more informative about Up. (User will need to log out and in again to get access to the new projects.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants