kelleydv / bapa Goto Github PK

This needs to be improved, by prefixing the proper host url (derived from config) as well as including a better message.

Officers need the ability to send email to all members. This is distinct from the sfbapgg Google Group, which is open to the public (and not all members want to be a part of it). This could be done via a mailer such as SendMail, although I'm not sure if it's meant to be used that way.

What would work best is if the site would automatically subscribe/remove people from the BAPA Google Group (https://groups.google.com/forum/#!forum/sfbapa - currently private since it's not working) according to their membership status (subscribe when they join/renew and remove/disable when their membership lapses).

Suggestions on licensing this code? One of the considerations in open-sourcing this was that other flying clubs could make use of it.

Start off with a page or two as an experiment to ensure we can satisfy all needs w/ Jade

There are some tests that cover the bare minimum, but they should be cleaned up and expanded.

The elephant in the room.

http://flask.pocoo.org/docs/0.10/testing/

ObjectIds are not being handled well in the pymongo abstractions. This needs attention. Need suitable object for session data, as well as db lookup.

The app sends a number of transactional emails. These currently have no markup or styling. They should be styled in some consistent and scalable way that allows adding additional emails without extensive work.

Currently, ushpa is the only login credential used. We probably want to let people register on the site even if they are not rated pilots.

When a user registers, the app hits the USHPA api and prevents the user from registering if there is an error.

This should be handled differently - for example, in case ushpa.aero is down.

Also, the data is being entered in the user record. The data may have different keys, depending on the type of pilot, etc. but MongoDB has "flexible schema". I'm questioning whether to move the pilot data to a separate collection, and so far answering no.

Comments welcome.

Configuration is specified by exporting an environment variable containing the path of the config with something like: export BAPA_CONF=config/default.cfg. Then the application loads the config from the environment variable.

The default configuration should be loaded automatically, and then updated with anything that may have been specified in a BAPA_CONF environment variable.