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

Update Eratosthenes to version 2 #76

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

Update Eratosthenes to version 2 #76

wants to merge 323 commits into from

Conversation

thngkaiyuan
Copy link

Migration Steps

  1. Merge this PR and update code on production server to version 2 code
  2. Add the line external_libraries_v2: /opt/codebender/codebender-external-library-files-v2 to Symfony/app/config/parameters.yml
  3. Update composer
  4. In Symfony directory, run app/console doctrine:migrations:migrate
  5. Clear cache

Cleaning Up

  1. Remove the line external_libraries: /opt/codebender/codebender-external-library-files from Symfony/app/config/parameters.yml
  2. Clean up old library files: rm -rf /opt/codebender/codebender-external-library-files /opt/codebender/codebender-arduino-library-files/libraries

Post-Migration

  1. Merge PR Remove old entities, fixtures and tables after migration #77 and update code on production server to remove the old entities and fixtures
  2. Run app/console doctrine:migrations:migrate to drop the old tables

thngkaiyuan and others added 30 commits February 24, 2016 14:51
@thngkaiyuan
Implement views toggleStatus
61ead11
@Gisonrg
Merge branch 'v2-api-development' of https://github.com/nus-fboa2016-…
4c164af 
…CB/eratosthenes into v2-api-development
@Gisonrg
consider if a library is disabled when checking external library
382a325
@Gisonrg
show files for all versions if no version given for fetch command
3b61533
@Gisonrg
Implement new view template for viewing libraries. Add v2 view templa…
fa0b9bb 
…te files
@bsmmoon
Version added on the form
2fec3c6
@bsmmoon
Changing namings
b92078d
@bsmmoon
More renaming
558c4a4
@bsmmoon
Changing saveNewLibrary
bf74755
@bsmmoon
Refactoring
418e695
@thngkaiyuan
Fix typo in method name
4499bfe
@thngkaiyuan
Remove magic branch
6366ee8
@thngkaiyuan
Correct another typo of getLiraryMeta
819b0d4
@thngkaiyuan
Implement isLibraryInSyncWithGit method and refactor code
d6c10c6
@bsmmoon
Added NewLibraryCommand
657653d
@bsmmoon
Version setters placed
0803170
@bsmmoon
Added ToDo
7fea9ec
@thngkaiyuan
Update search action for v2 API
9cafa6a
@bsmmoon
Naming conflict resolution. Refactoring.
f593930
@bsmmoon
Refactoring. Renaming.
3f97f93
@thngkaiyuan
Turn fetchLibraryFiles and fetchLibraryExamples public
81cdca9
@thngkaiyuan
Implement v2 of downloadAction
558170a
@thngkaiyuan
Refactor common methods into API handler
d7b6376
@Gisonrg
Fix the broken search UI
2f8dc5d
@Gisonrg
Add more test cases to strengthen test coverage
e0172ab
@bsmmoon
Fixing api list. Needs clarification on LibraryExample entity.
5c85eca
@bsmmoon
Passed the test
6a45e68
@bsmmoon
Putting the structure. Needs to clarify schema design.
b58653a
@bsmmoon
Error is gone. Needs to refine the output and make test.
62dbe7a
@bsmmoon
Finalized. Test added.
2f53ab1
bsmmoon and others added 23 commits April 6, 2016 11:46
@bsmmoon
Merge branch 'remove-duplicate-codes' into v2-merged-2
3839dd4
@bsmmoon
Merge remote-tracking branch 'origin/v2-merged-2' into v2-merged-2
725844f 
Conflicts:
	Symfony/src/Codebender/LibraryBundle/Handler/ApiCommand/ListApiCommand.php
@bsmmoon
List to handle the current directory dot
8bfbc03
@bsmmoon
Merge branch 'remove-duplicate-codes' into v2-merged-2
56839d9
@bsmmoon @thngkaiyuan
Changed name to follow the naming convention
97cef16
@thngkaiyuan
Change getExample API error messages
fb277de
@thngkaiyuan
Change error message for getKeywords API
2cf29fd
@thngkaiyuan
Change getExample API error messages again
d196080
@bsmmoon
Delete Command implemented. Handling of new latestVersion missing.
b881aaf
@bsmmoon
Fixtures added for testing. Tests added. Needs permission fix.
247dee4
@bsmmoon
Better modularization.
916daee
@bsmmoon
More testing cases
5baefac
@bsmmoon
Added message on successful delete.
4355d8c
@Gisonrg
Add deletion button in the viewLibrary page
e1902d0
@thngkaiyuan
Fix bug in deleteLibrary API
eb51959
@thngkaiyuan
Rename delete to deleteLibrary for clarity
d1d79a1
@bsmmoon
Latest version handling added.
775ba95
@bsmmoon
Separated database handling from other logic for safety. Can fail mor…
180b7b8 
…e safely without corrupting the database.
@bsmmoon
Added new tests for deleting a latest version. New fixtures added for…
c6c98b7 
… the tests.
@thngkaiyuan
Implement alternative way of changing latest version in delete api
63cfe4e
@thngkaiyuan
Fix a bug and set the last commit of the library after changing the l…
11e486b 
…atest version
@thngkaiyuan
Add some test documentation
84b0f29
@thngkaiyuan
Merge branch 'master' into v2-merged-2
02a47b3
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 02a47b3 on v2-merged-2 into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 27e442f on v2-merged-2 into * on master*.

@thngkaiyuan thngkaiyuan mentioned this pull request Apr 12, 2016
Open
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
Reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@thngkaiyuan @coveralls @spiliot @Gisonrg @bsmmoon