Skip to content
This repository has been archived by the owner on Jul 21, 2020. It is now read-only.

Collections backend #890

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Collections backend #890

wants to merge 3 commits into from

Conversation

BenHenning
Copy link
Owner

This branch introduces the backend functionality needed to support collections.

BenHenning and others added 3 commits July 21, 2015 13:20
Changes in this commit include:

- Introduce the domain object for collections.

- Introduce the domain services for collections, including creating,
  updating, and deleting collections.

- Introduce the GAE storage model for collections.

- Implement many tests for verifying state and functionality in both the
  collection domain object and its services. Many of these are derived
  from related tests for explorations.

- Introduce test utility functionality for creating new collections.

- Implement search functionality for collections.

- Implement rights management for collections distinguished from
  explorations.

- Introduce demo collections (as a concept) and the first demo
  collection.

- Begin the schema versioning for collections.

- Introduce event handler for recording collection content or status
  changes.

- Update the UserSubscriptModel to store collection IDs.

- Implement subscriptions for collections.

- Various aesthetic and comment cleanups as seen throughout the
  codebase.
@BenHenning
Copy link
Owner Author

Crap!

@BenHenning BenHenning self-assigned this Aug 9, 2015
@BenHenning
Copy link
Owner Author

Test


We suggest you follow the method using .netrc, since it is more convenient. After editing your ~/.netrc file, run

git clone https://code.google.com/r/[YOUR_REPO_NAME]/
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

COMMENT WHY YOU DELETE?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

REPLY

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New line note.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reply to a comment as a non-collaborator.

@BenHenning
Copy link
Owner Author

Pull request-level comment.

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

Successfully merging this pull request may close these issues.

2 participants