Skip to content
This repository was archived by the owner on Jan 28, 2020. It is now read-only.

Conversation

pwilkins
Copy link
Contributor

rc/0.6.0

George Schneeloch and others added 30 commits July 20, 2015 12:04
Implemented learning resource panel updating on every panel open
Used different haystack index for tests to prevent conflict with web application
Prevents any funny business with special characters. Incidentally saves
space in Elasticsearch.

Fixes #384
Replaces and closes #374.
Replaces and closes #316.
Changed vocabulary term indexing from string to integer.
Implemented preview link for learning resource panel
- Removed automatic index update from deployment
Fixed preview link not showing up in list view
…repo_view

Refactored permission check for listing view
Refactored code for reloading module into a function
Removed export view which isn't used anymore
Extended tests for manage_taxonomies.jsx file
Added docker support for running worker or Web process by environment
@amir-qayyum-khan
Copy link
Contributor

@pwilkins can you remove this
[] Extended tests for manage_taxonomies.jsx file. Amir Qayyum Khan
from this PR,
Because i cant test it here, It contains all unit tests for manage_taxonomies.jsx

cc @pdpinch

@bdero
Copy link
Contributor

bdero commented Jul 31, 2015

@amir-qayyum-khan If it's all unit tests, I would maybe just check if the coverage report for this build (https://coveralls.io/builds/3186919) shows coverage on the lines that your tests verify.

@amir-qayyum-khan
Copy link
Contributor

@bdero it is fine. Coverage is equal to what I have on my local machine.

@pdpinch what do you say?

@noisecapella
Copy link
Contributor

The export button is not showing up for a particular non-empty repository, I'll need to investigate that bug.

@noisecapella
Copy link
Contributor

It may be easiest just to revert that commit and fix it after the release, the commit is only two lines. Thoughts?

@ShawnMilo
Copy link
Contributor

Vocabulary term indexing by primary key (integer) instead of slug confirmed. I marked "avoid hitting the database on the search page" complete because it is covered in unit tests and there's nothing to "see" on the CI site.

George Schneeloch and others added 3 commits July 31, 2015 13:47
Fixed export button not appearing in certain situations
- Removed export button for empty repositories.
- Added test for StaticAsset.loader.
- Added export functionality for learning resources.
- Added select2-bootstrap-theme bower component.
- Added Select2 to the JS libraries.
- Created ICheckbox React component.
- Made XML preview box for a LearningResource should be read only.
- Pinned all versions.
- Avoided hitting the database for the search page.
- Added field to Vocabulary to define if it can contain multiple terms.
- Incremented xbundle version.
- Added test for ManageTaxonomies.loader.
- Changed vocabularies listing page to match the design.
- Fixed broken links in the footer.
- Removed console.error statement.
- Fixed bug where export checkboxes were not updated in sync with
export count.
- Fix test failures due to pylint dependency chain.
- Created StatusBox component to hold messages and errors.
- Added shopping cart for export.
- Changed response vocabulary name to match input and avoid key collision.
- Added docker support for running worker or Web process by environment.
- Extended tests for manage_taxonomies.jsx file.
- Added description path to listing page.
- Removed export view which isn't used anymore.
- Refactored code for reloading module into a function.
- Refactored permission check for listing view.
- Updated Haystack to 2.4.0 - Removed automatic index update from deployment.
- Fixed preview link not showing up in list view.
- Grouped REST tests by common endpoint.
- Changed vocabulary term indexing from string to integer.
- Implemented preview link for learning resource panel.
- Added sorting to search results.
- Implemented learning resource panel updating on every panel open.
- Used different haystack index for tests to prevent conflict with web
application.
pwilkins added a commit that referenced this pull request Aug 3, 2015
@pwilkins pwilkins merged commit 8978469 into release Aug 3, 2015
@bdero bdero deleted the rc/0.6.0 branch August 19, 2015 12:56
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.

9 participants