Skip to content
This repository has been archived by the owner on Jun 1, 2022. It is now read-only.

API for everything #261

Merged
merged 68 commits into from
Jul 28, 2015
Merged

API for everything #261

merged 68 commits into from
Jul 28, 2015

Conversation

ptgolden
Copy link
Member

This PR:

  • Adds in information about items that was previously missing in the API in several places
  • Adds a way to include links with rels in JSON responses
  • Removes the frontend application, so that this application will only handle JSON API requests
  • Removes HTML rendering altogether

TODO:

@ptgolden
Fix serialization issues with scans
608a8e7 
Also, add width/height values to their representation. That might be
removed, but it made creating the leaflet.js interface easier.
@ptgolden
Change document view- remove tabs; make new scan interface
1f21562 
Scans are handled with leaflet.js instead of Seadragon now

This substantially changes how document pages look now- there's a lot of
work left to do on it still.
@ptgolden
Add 'cited_by' field to document resource
8d5ac68
@ptgolden
Use API serializer to render main document view
f40a508
@ptgolden
Use content negotiation to serve API
466b789 
This, rather than starting every URL for the API at the /api/ root
@ptgolden
Refactor views and serializers; add new user serializer and view
3b0d22f
@ptgolden
Fix note view permissions
aaa95fc 
So that unauthenticated users are able to view notes that are not
private
@ptgolden
Render note template from API data
536d4e0
@ptgolden
Fix typo in document template
365e9af
@ptgolden
Fix API tests to use content negotiation
65bd0c3
@ptgolden
Fix various URL errors
15045cf 
All related to collapsing API and main url schemes
@ptgolden
Fix bug in note serializer
4812e86 
ValidationError wasn't being raised when there was something wrong with
the nested sections
@ptgolden
Fix typo in API tests
a020c24
@ptgolden
Make license and updaters field on note serializer read-only
b676bfb
@ptgolden
Adjust URLs to note use "/api/" in the client application
e6ad553
@ptgolden
Fix suffixes for urls
99dda3d 
.json/.html/.api replace trailing slashes in URLs now, instead of
following them
@ptgolden
Start using elasticsearch_dsl for queries; add pagination
5e254e2
@ptgolden
Add permissions field to note serializer
d8ee022
@ptgolden
Fix a few bugs in document/citation serializers
7be38bc
@ptgolden
Render note template from API data
db05414
@ptgolden
Serializers/views, for project roles and featured items
02f0d50
@ptgolden
Fix bug in root api view
23583f7 
add "format" kwarg
@ptgolden
Read-only "Browse" API view
ecddcf8 
Could be removed later if it's not appropriate
@ptgolden
Fix typo in requirements.txt
68da6d6
@ptgolden
Bump django rest framework to 3.1.0
5b0a7e8
@ptgolden
Overhaul pagination and linking
bf9dc5a 
This commit adds the concept of links to resource views, and makes
pagination work with them. For now, only implemented in Note list and
retrieve views.
@ptgolden
Make sure last_updated field is properly mapped in elasticsearch
eb01b07
@ptgolden
Add add/edit/delete links to document and topic pages
4ad0979
@ptgolden
Add tests for hyperlinks in json responses
9f39152
@ptgolden
Support root URL without requiring a slash
90d5e2f 
I'm not exactly sure what the deal was with this commit, it was all
written several months ago. I'm pretty sure it was necessary.

(sorry)

Regardless, a lot of this is going to get bombed away when we take HTML
rendering out of the API.
@ptgolden
Install nose, django-nose
a200d01
@ptgolden
Show more helpful EnvironmentError when ElasticSearch is not running
8f443fe 
Fixes #259
@ptgolden
Use django-nose as test runner
60f629e 
also, fix some broken URL reversals (I didn't mean to, but I
accidentally amended the commit)
@ptgolden
Rename some tests to make them work with nose discoverer
2cab847 
Also, fix a few *more* link reversals. Sorry! It's late and I'm tired.
@ptgolden
PEP-8 on API test
13d179d
@ptgolden
Remove NPM things from .travis.yml
abde0b1
@ptgolden
Adopt some more tests to django_nose
9b3b2e1
@ptgolden
Update travis test command
a35e8c3
@ptgolden
Make auth module; move auth models to it
b9016a2
@ptgolden
Remove static styles directory
777c80a
@ptgolden
Remove base templates
2ac37e7
@ptgolden
Remove more unused code
e1793fd 
Static files (images, css, js), templatetags, templates, requirements...
@ptgolden
Set up basic authorization section homepage
01d5653
@ptgolden
Add shell for auth pages
03813c4
@ptgolden
Add logged in user to auth homepage
f8025ef
@ptgolden
Create sign in/out templates
6b4e2d0
@ptgolden
Add signout link to navbar
2c2bae0
@ptgolden
Add templates for account creation and password reset
2c87e6c
@ptgolden
Implement account creation/activation workflow
5c0c197
@ptgolden
Remove custom UserManager functionality
4da969a
@ptgolden
Add shell pages for profile, account, project pages
6b9292f
@ptgolden
Get create -> confirm workflow working
ae891c5
@ptgolden
Allow editing profile information
c17f3ca
@ptgolden
Enable adding new projects
837bd69
@ptgolden
Add basic project settings page
d26d0cb
@ptgolden
Expand shell of user account settings page
a2a95e3
@ptgolden
Copy link
Member Author

Merging this so that work can go on master. All the things I pointed out in the original comment will move to an issue/milestone

ptgolden added a commit that referenced this pull request Jul 28, 2015
@ptgolden
Merge pull request #261 from editorsnotes/api-everything
e37756a 
Use API for all data except auth; get rid of HTML interface (except auth)
@ptgolden ptgolden merged commit e37756a into master Jul 28, 2015
@ptgolden ptgolden deleted the api-everything branch October 19, 2015 17:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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
@ptgolden