systers / macc Goto Github PK

Fix logout button so that it's still visible on small screens and windows.

Currently, the project uses a local copy of bootstrap on the server side instead of Bootstrap CDN.

Avantage of using the Bootstrap CDN:
Many users already have downloaded Bootstrap from MaxCDN when visiting another site. As a result, it will be loaded from cache when they visit your site, which leads to faster loading time. Also, most CDN's will make sure that once a user requests a file from it, it will be served from the server closest to them, which also leads to faster loading time.

I propose to add CDN by adding few lines in the html pages and also keep the local copy in case of a fallback.
More can be read here - http://stackoverflow.com/questions/26192897/should-i-use-bootstrap-from-cdn-or-make-a-copy-on-my-server
@medhach @Neetuj Should I send a PR?

Unit tests will help improve the application performance.

Is this functionality needed? There are no views or serializers for the same. In fact you cannot even create any contact. Is this supposed to stay or shall we remove this completely?

Bootstrap or further improvement in UI ?

• Adding material fonts

• separate css folder.

• bootstrapping views

Currently the field types are as follows

1. title - CharField
2. phone - CharField
3. location - CharField
4. gender - CharField

I want to change the field types and add regex validators too. Then it would be easier to write tests for user as well.
Also, I think it would be nice if there were utility tests added which could create posts, users and pcusers on their own. We can use them at once and just add mock data rather than writing everything again and again. @medhach Can I go ahead?

When the user logs in for the first time, and has not set his/her name the welcome button displays "Welcome, " which looks incomplete . Instead it should display something like "Welcome, User" if the until the user has not set his/her name.
@medhach I would like to send a PR.

Try to get a relevant picture to add in the Dashboard for PCSA app

Right now, gender is entered in a text field. Gender should be selected from a dropdown (or radio buttons) instead.

Clicking on the Cancel button in the Edit Profile form will submit the form (instead of canceling).

Currently no contact can be saved. The same has to be implemented via pcsa_GHN app. @medhach Can I work on this? :)

This way we can check several things related to the profiles. @medhach Shall I continue?

Traceback (most recent call last):
File "/usr/lib/python2.7/wsgiref/handlers.py", line 86, in run
self.finish_response()
File "/usr/lib/python2.7/wsgiref/handlers.py", line 128, in finish_response
self.write(data)
File "/usr/lib/python2.7/wsgiref/handlers.py", line 212, in write
self.send_headers()
File "/usr/lib/python2.7/wsgiref/handlers.py", line 270, in send_headers
self.send_preamble()
File "/usr/lib/python2.7/wsgiref/handlers.py", line 194, in send_preamble
'Date: %s\r\n' % format_date_time(time.time())
File "/usr/lib/python2.7/socket.py", line 324, in write
self.flush()
File "/usr/lib/python2.7/socket.py", line 303, in flush
self._sock.sendall(view[write_offset:write_offset+buffer_size])
error: [Errno 32] Broken pipe

Rename infohub (the app that contains project settings) to macc, since MACC is the name of the project.

When user is logged out and inputs a URL that requires login credentials, user is taken to the login page. After logging in, however, the application does not navigate to the intended page.

It improves the quality of the page. :)

For better cohesiveness and less coupling.

In various forms there are certain problems. eg in edit profile form-

1. 'First name' field can be left empty, it should be made mandatory.
2. 'Mobile Number' allows text too. It should be limited to numbers.
3. 'Email' field allows invalid emails.
   @medhach I would like to add validations to these forms and send a PR.

Rename malaria to malaria_web to make it more clear that the code in this Django app implements functionality to be accessed on the user's browser (and is not API code).

For better cohesiveness and less coupling.

When the application is installed for the first time, the user creates a superuser and is required to create a Pcuser in the Django admin interface (that corresponds to the superuser). This should be done automatically.

The welcome and logout buttons are not buttons, but rather hyperlinks with borders. I would like to change then to button of appropriate types.
@medhach I would like to work on this.

The title_change and description_change boolean fields in the RevPost model are no longer required/being used so they should be removed from the RevPost model.

Input validation is currently not allowing the ' character in the Post description field.

We can add automated tests for the app as well. Can someone please approve for the same?

This will let us add some compulsory features for various Pcuser fields and regex validations. @medhach
Can I go ahead with it?

Can I take up the task to make this page better too?

The templates have the css styling embedded in them. This causes a lot of problems in finding out common glitches as well. Also, #22 and #21 are also related to the same problem. I want to create an entire independent styling structure and resolved from the templates. @medhach Is this worth it?

There are 2 problems with the installation guide-
1.The instruction to be followed when not using the Peacecorps VM are a bit unclear and can be improved. The document clearly explains the approach which uses the Peacecorps VM, but the one without it gets lost after the third step. I myself faced a bit of difficulty while using the second approach.Hence, I would like to add instructions about which steps to skip and what extra steps to take when using python's own virtual environment after the third steps of the document.

2.The URLs to be entered in the browser are given as http://localhost:8001 and http://localhost:8001/admin, where as they should be http://localhost:8000 and http://localhost:8000/admin respectively.
@medhach @desaivaibhavi I would like to send a PR for this.

Separate web app Malaria functionality and Malaria API functionality for better cohesiveness.

Write tasks for Google Code-In 2015 (https://developers.google.com/open-source/gci/?hl=en) which Systers will be participating in. Assigning this to myself since I'm planning on being a mentor.

Peace Corps wants an additional field so that they can add an article or link from an external source. It would be nice to have them enter a URL in a new field (maybe call it "external resource", we can get the confirmed label name from them) and then if there is a default image associated with the article, we post it like a Facebook post or Twitter post.

@Nerdylicious

Separate web app PeaceTrack functionality and PeaceTrack API functionality for better cohesiveness.

• webhub

• pcsa

• pcsa_GHN

For better cohesiveness and less coupling.

Steps to fix.

1. export DJANGO_SETTINGS_MODULE="infohub.settings"
2. then run python manage.py runserver

To Do:

Need to implement in such a way that we do not have to import everytime.

Fix profile button so that it's still visible on small screens and windows.

Add input validation for the following fields:

• First Name
• Last Name
• Mobile
• Email
• Gender
• Location

Input validation includes not allowing certain special characters, having a maximum length for each field and following a specific format/regex for applicable fields (for example, the phone number field).

Tablet View:-

vs.

Desktop View:-

How is this design for the homepage? Can I send a PR to make it look like this? @medhach :)

The Profile and Logout links could be added here.

@MeepyMay @AngieSky
Since this repository is for the project Mobile App Control Center, I think we should change the name so it can be more apt and easy to find.

Fix the change password functionality so that it checks that the New Password field and Confirm Password field match. Currently, the form is not checking whether the input to these 2 fields match.

The link to Installation Guide given on the README.md does not point to the Installation guide. It rather opens up the API documentation. Also, API documentation link does not point to the API documentation given in the project, but to a google doc guide. I would like to send a PR fixing this @medhach @desaivaibhavi .

Move styling into CSS files instead of embedded in HTML to separate the content (HTML) from the presentation (CSS).

To test proper functioning of the app.

I was thinking that while installation guide is already in a steady process to get updated, we might also add a separate 'Contribution' file explaining and guiding the people for the following:

1. Adding remote and upstream
2. Git branches and rebasing
3. Squashing commits and force pushing the changes.

@Neetuj @medhach I would like to send a PR for this. Can I go ahead ? :)