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.

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

feat(annotationLibrary) Adding annotation comments on annotation item page #118

Merged
merged 12 commits into from
Dec 12, 2014

Conversation

cofiem
Copy link
Contributor

@cofiem cofiem commented May 12, 2014

No description provided.

Mark and others added 12 commits May 12, 2014 17:49
Also added stubs for creative directive for comments.
Also added stubs for creative directive for comments.
…ics/baw-client into annotation-comments

Conflicts:
	src/app/annotationLibrary/annotationItem.tpl.html
	src/app/annotationLibrary/annotationLibrary.js
- Adpated annotation comments to the new API format
- Converted links to buttons where appropriate
- Fully implemented reporting comments functionality (and reported warning icon / font color)
- Ensured basic form validation was working.
- Fixed the date formatting for comments
- Simplified and refactored the forms
- Added functionality to bawResource so that it does not expect an array result for `query()` calls
@atruskie atruskie changed the title [WIP] Adding annotation comments on annotation item page [Review] feat(annotationLibrary) Adding annotation comments on annotation item page Dec 12, 2014
@atruskie atruskie changed the title [Review] feat(annotationLibrary) Adding annotation comments on annotation item page feat(annotationLibrary) Adding annotation comments on annotation item page Dec 12, 2014
atruskie added a commit that referenced this pull request Dec 12, 2014
feat(annotationLibrary) Adding annotation comments on annotation item page
@atruskie atruskie merged commit 99c3397 into master Dec 12, 2014
@atruskie atruskie deleted the annotation-comments branch October 26, 2015 01:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants