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.

Sign up for GitHub

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

Jump to bottom

Versioning support #107

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Versioning support #107

wants to merge 29 commits into from

Conversation

u8sand
Copy link
Contributor

@u8sand u8sand commented Dec 11, 2018 •
edited
Loading

Fixes #43
Fixes #65
References #28
References #37
References #46
References #9

Still need to:

  • Thoroughly test and ensure it will seamlessly move to production
  • Wait till after Slugs #106 which was essentially taken out of this work
  • Merge into recent changes (large branch divergence)
  • Merge database migrations

u8sand added 29 commits September 7, 2018 17:03
@u8sand
WIP: Version controlled database models
eb10887 
- cleanerversion dependency installed
- database migrations created (not complete)
@u8sand
WIP: Version controlled database models
6cc6fcd 
- cleanerversion extended:
  - VersionManager--`use_in_migrations`
@u8sand
Versioned migrations
fd7b5b3 
- VersionedQuerySet extension
- Override create to force certain methods on resulting objects

Fixes #43
@u8sand
Completed migrations to new Database
c84a652 
Fixes #43
@u8sand
objects -> objects.current
13432c9 
CustomModelViewSet `get_queryset` fix
@u8sand
WIP: refactoring of FAIRshakeAPI.views for version controlled assessment
c81ce76 
References #43
References #46
@u8sand
Base64-serialize uuid fields
6c57718
@u8sand
Enabled multiple urls to be specified
c1a5f98 
TODO:
- make field larger
- ensure they can be queried independently
@u8sand
Added cache table directions to README
9fe1bd4
@u8sand
Added slug fields (defaulting to uuid)
48a3d44 
Fixes #65
@u8sand
Merge branch 'master' into versioning
38c7ef5
@u8sand
Made FAIRsharing automated assessment use DOI
766d729 
References #37
References #28
References #9

- Removed fairsharing field (now determined from doi url)
@u8sand
Removed author name search vector
dc75488 
Not that necessary and causes an issue with cleanerversion
@u8sand
Added quotes to data-target ids--now slugs
bb8aaa6
@u8sand
Ensure slugs don't collide with object ids as that will be a fallback
0d88c51
@u8sand
Added coverage and information on running tests
e29bc8b
@u8sand
Wrote some FAIRshake API unit tests
c63b084 
- Test database setup
- Test all viewset-list endpoints (html & json)
- Test all viewset-retrieve endpoints (html & json)
@u8sand
Wrote some FAIRshakeHub unit tests
ff01228 
- Use FAIRshakeAPI's test database
- Test all standard views and search
- Test project stas view with all plots
@u8sand
Permissions fix for Assessment list/retrieve
affc675
@u8sand
CustomUUIDField handle None value
cad977f
@u8sand
Fixed VersionQueryEx edge case (instance is str)
7ae2bb0
@u8sand
Moved CustomRouter to extensions.rest_framework_ex
96fd16c
@u8sand
Fixed assessment object query model names
6e0d1d5
@u8sand
Merge branch 'master' into versioning
b60a90f
@u8sand
Merge branch 'master' into versioning
d72de8a
@u8sand
Merge branch 'master' into versioning
8aeac02
@u8sand
Added unique_together constraints on assessment and answers
8983efc
@u8sand
Merge branch 'master' into versioning
359bf97
@u8sand
Merge branch 'master' into versioning
448ef7c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@u8sand