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

Develop #453

Closed
wants to merge 171 commits into from
Closed

Develop #453

wants to merge 171 commits into from

Conversation

Zaknarfen
Copy link
Contributor

No description provided.

Zaknarfen and others added 30 commits September 14, 2018 22:42
	- Moved the force service to the core module
	- Moved all PrimeNG references to the shared module
	- Refactored the shared module imports for other
	modules
	- Added two fields to the team member model
	- Changed the way that the contribute teams page load
	all the images, before we had to wait for requests and
	now we make all the requests in parallel
	- Related to #375
	- Changed the way the images are loaded in the contrib
	page, now they load async
	- Allowed the contributing teams page to gradually render
	images that are downloaded, and using a placeholder while
	waiting
	- Related to #375
	- Fixed a resizing error in the genes list component
	- Moved more methods to the api service
	- Changed calls in several components to use the new api
	methods
	- Moved all the loading logic into the a new data service
	method
	- Added parallel loading to the data service when loading
	chart data
	- Closes #380
	- Closes #381
Change badge to travis-ci.org
	- Removed justification from text in the intro
	page
	- Changed modals look to match the Figma designs
	- Closes #382
	- Closes #383
	- Fixed wrong colors and icon position in the
	multiselect component
	- Fixed wrong colors and icon position in the
	checkbox component
	- Added new images to fix design issues
	- Added an image for the nominated gene button,
	synapse account creation
	- Added a plus and minus icons
	- Fixed an issue with mobile modals
	- Closes #386
	- Closes #388
	- Closes #389
	- Closes #391
* 'develop' of github.com:Sage-Bionetworks/Agora:
  Fixing more styling issues
	- Fixed a filter problem with the selectMenu
	chart when loading the same gene twice
	- Fixed left space clipping values in the forest
	plot
	- Added the ability to load the same gene from list
	or search without reloading the data
	- Changed the gene overview tests to reflect the new
	functionalities
	- Changed the gene service and gene service stub to
	reflect the changes
	- Closes #395
	- Closes #403
	- Closes #406
	- Moved icons to the icons asset folder
	- Changed the default checked icon to the image
	used in the Figma design
	- Fixed the incorrect Y padding with the box
	plot chart
	- Added a link from Druggability in the Summary of
	Evidence to the Next Steps section of the page
	- Added a new package, ngx-page-scroll
	- Closes #381
	- Closes #401
	- Fixes a bug when reloading the same gene using the
	browser history button
	- Closes #403
	- Fixed a loading issue preventing information to get
	rendered properly
	- Changed a few api endpoints return values that were
	faulty
	- Changed a few design things related to the network widget
	- Fixed the column position for the network widget when
	there is no expression data
	- Closes #414
* Setup automated docker container deployments to the official
Sage dockerhub repository at
https://hub.docker.com/u/sagebionetworks/dashboard/
* Split CI builds into separate stages
* Setup Agora tests to run against the built and deployed agora
containers in dockerhub 'sagebionetworks' org.  This validates
the container builds and deployments.
* 'develop' of github.com:Sage-Bionetworks/Agora:
  Docker deployments to dockerhub
	- Added basic tests to several components
Create a separate CI stage for tests.
	- Removes the end of the text after RefSeq, or
	variances in the genes description
	- Added a inner section divider so prevent
	user confusion
	- Fixed an incorrect text display for the
	nomination of a target gene in the gene network
	component
	- Added visual feedback when searching genes
	- Added a new response interface type
	GeneInfosResponse
	- Changed the info icon to use the new design
	in Fimga
	- Fixed an issue with the BETA text not being
	bold
	- Updated vulnerable packages
	- Bumped data version to 8
	- Closes #412
	- Closes #413
	- Closes #425
	- Closes #428
	- Closes #429
	- Related to #433
Starting nginx container in a t2.large EC2 would result in error

  "20000 worker_connections exceed open file resource limit: 1024"

Update nginx configs to allow it to startup without error.
	- The teams page now shows teams in alphabetical
	order
	- Capitilized the Y axis label in the box-plot
	- Related to #375
	- Closes #395
	- Closes #435
* 'develop' of github.com:Sage-Bionetworks/Agora:
  Adding alphabetical order to the teams page
  tune nginx for AWS

# Conflicts:
#	src/app/genes/gene-details/gene-similar/gene-similar.component.ts
	- Added a hotfix to the teams ordering, images were correct
	but teams weren't
Zaknarfen and others added 29 commits October 31, 2018 20:01
conditionally build client according to environment
This caused invalid yaml exception.
This reverts commit a0c6a78.
conditional environment builds of agora
* 'develop' of github.com:Sage-Bionetworks/Agora:
  fix conditional build on beanstalk ec2
  Revert "Revert "conditionally build client according to environment""
* 'develop' of github.com:Sage-Bionetworks/Agora:
  Small fix removing linebreak from local download script
	- Added X axis labels to the box plot
	- Added a new tooltip to the brain tissues
	in the X axis on the forest plot
	- Added the model above the forest-plot
	- Closes #556
	- Removed the duplicated charts
	- Removed duplicated tooltips
	- Closes #558
AWS support says that the script is trying to delete a file that
that doesn't exist.  Their suggestion is to ddd some logic to only delete
if file exists.
Attempt to fix nginx retart error
This time AWS support suggests using initctl instead of service
command.

FROM AWS SUPPORT:
 I have done some research on the issue and have found that it is an
 ongoing issue with the node.js environment and the cause for the issue
 all node.js environment greater than equal to 4.4.6 version is
 reporting an error in service nginx restart.  I have found that instead
 of service command they are now using the initctl for starting and
 stopping the nginx service.
another attempt to fix nginx restart
command arguements need to be in correct order
another attempt at nginx restart fix
	- Changed embed video from YouTube to Vimeo
	- Updated Differential Expression page layout
	to match Figma design
	- Closes #541
	- Closes #557
@Zaknarfen Zaknarfen closed this Nov 5, 2018
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.

5 participants