Skip to content

Fixes for alpha tester #748

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

Merged
merged 58 commits into from
May 8, 2019
Merged

Conversation

odeimaiz
Copy link
Member

@odeimaiz odeimaiz commented May 6, 2019

What do these changes do?

  • More logs in node_ports and sidecar
  • New GH issue has now two labels and it is assigned to oSPARC Backlog project
  • More logs in node_ports and sidecar
  • ISAN template fixes

image

sanderegg and others added 30 commits March 26, 2019 09:41
…icies

Revert "Feature/permission policies"
@ghost
Copy link

ghost commented May 6, 2019

DeepCode analyzed this pull request.
There is 1 new info report.

Click to see more details.

@odeimaiz odeimaiz changed the title WiP: Some fixes for alpha tester Minor fixes for alpha tester May 6, 2019
@codecov-io
Copy link

codecov-io commented May 7, 2019

Codecov Report

Merging #748 into master will increase coverage by 0.27%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #748      +/-   ##
==========================================
+ Coverage   65.24%   65.51%   +0.27%     
==========================================
  Files         120      113       -7     
  Lines        4785     4533     -252     
==========================================
- Hits         3122     2970     -152     
+ Misses       1663     1563     -100
Impacted Files Coverage Δ
api/tests/test_conventions_openapi.py
api/tests/test_individual_json_schemas.py
api/tests/conftest.py
api/tests/test_individual_openapi_schemas.py
api/tests/test_against_openapi_core.py
api/tests/test_full_openapis.py
api/tests/utils.py

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0dfbd40...0d4f828. Read the comment docs.

@odeimaiz odeimaiz changed the title Minor fixes for alpha tester Fixes for alpha tester May 7, 2019
Copy link
Contributor

@ignapas ignapas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cannot download a file and the deleting button causes the tree to collapse. The design and structure of the FileLabelWithActions widget could be improved. But I think that is out of the scope of this PR.
data-manager

@odeimaiz
Copy link
Member Author

odeimaiz commented May 7, 2019

Cannot download a file and the deleting button causes the tree to collapse. The design and structure of the FileLabelWithActions widget could be improved. But I think that is out of the scope of this PR.
data-manager

Downloading/Uploading doesn't work in local environment. And yes, FileLabelWithActions can be improved, we will tackle it together with its backend.

@ghost
Copy link

ghost commented May 7, 2019

DeepCode analyzed this pull request.
There are 3 new info reports.

Click to see more details.

@ghost
Copy link

ghost commented May 8, 2019

DeepCode analyzed this pull request.
There are 3 new info reports.

Click to see more details.

@odeimaiz odeimaiz merged commit 4037c5e into ITISFoundation:master May 8, 2019
@odeimaiz odeimaiz deleted the feature/alpha-tester branch May 21, 2019 09:37
sanderegg added a commit that referenced this pull request May 22, 2019
* Delete settings.json

* Fixes for alpha tester (#748)

* fake-template-projects.json created. 2 templates moved here
* Kember use case added
* MattWard use case added
* 4x CC 0D added
* Provide a way to refetch data from DataManager
* connect the kember-model to the kember-viewer
* moved null handler to node_ports package
* set node ports log level to same as sidecar
* removed Unused import logging
* New issue: add UX_improvemnts label and asign it to oSPARC Product Backlog project

* Permission policies (#731)

* LinkButton and LinkLabel moved
* Project -> Study
* default hardcoded permissions added
* Added some restrictions
* Link -> Edge
* Moved permission checks to data level
* Set Default Role
* more permission checks
* token permission checks
* datcore read permission check
* study.node.data.* permission check
* study.node.rename permission check
* services.filtered.read permission check
* fixed plotly minor bug
* Test permissions
* Moved some removeNode code to Workbench
* Moved removeEdge code to Workbench
* test edge create/delete
* test node rename
* deserializeNodes and deserializeEdges added
* deserializeEdges fixed for 'outputNode's
* got rid of canStart
* Typo: presgined -> presigned
* do not allow anonymous users to add/delete tokens
* do not allow anonymous users to update user profile

* Frontend privileges. 2nd iteration (#753)

* help points to our itisfoundation.github.io
* typo
* Do not show Debug level in logger to User and Anon
* Do not show nodeIds in NodesTree to User and Anon
* Tester and higher level roles can switch their role to a lower one

* Is730/serve studies (#747)

Implements a first prototype for issue #730 

Adds new subsystem ``services/web/server/src/simcore_service_webserver/studies_access.py``
 - Open services/web/server/tests/sandbox/*.md and click any link
 - If user is logged in, it copies the project and displays it
 - if anonymous, it creates a temporary user and does the previous step
 - Merges changes from PR #734 that contains the UI part.
   - queries id in url and opens page with specified study
   - fixes log out
   - preview of permissions (client-side only)

Next step will incorporate privileges and user roles on the server side.

* Quick fixes (#742)

A couple of iframe bugs fixed:

Maximizing an iframe now works.
We can now resize a SidePanel over an iframe
Other bugs:

File picker bug fixed. #741

* Update study_access_demo.md

Added link to actual emulator [skip travis-ci]

* Logger context dependant (#754)

* replace label by nodeId in logger
* Bring back the update page alert window
* added link to version/commit to login page
* startInteractiveNode out from populateNodeData
* Log message changes when label is changed
* Logger node context dependant

* Is234/compile requirements (#733)

Implements #234 : normalizes dependencies for python packages in osparc-simcore repository

- documented (see ``docs/python-dependencies.md``) new workflow with python requirements
- updates ``packages`` and ``services`` 
- updates ``ops\travis`` CI
   - moves out a part of unittest ``simcore-sdk`` into a new unit-test for ``service-library``
   - installation is always performed as ``pip install -r requirements/ci.txt ``
   -  @mguidon,  sidecar as NO tests (created ``ops\tracis`` but disabled)!

* Fixed makefile for all OS
* Fixing storage tests:
 - downgraded aiopg until bug is fixed
* Fixes overlapping port
* Enhaces doc
* Downgraded aio-pika
 - avoid test-errors in webserver integration!
 - cleanup

* Fix input node's title (#771)

* fix #770

* Improve CI/CD for simcore and add staging/production stages (#738)

build docker images once and push to dockerhub
pull from dockerhub for testing
implement release workflow

* Is788/allow different stack name (#789)

* director/sidecar independant of stack name using SWARM_STACK_NAME
* allow choosing port where simcore platform is available

* fixed link to image

* Maintenance/update requirements (#792)

* Upstead api/tests requirements
* Updates travis system-testing requirements
* Updated s3wrapper requirements
* update servicelib requirements
* Updates simcore-sdk requirements and with this all packages requirements
* Updates apihub/tests requirements
* Fixes missing ops/travis requirements
* Updates director
* Updates sidecar
* updated storage

* Fix: 2 file management bugs (#782)

* getLowerRoles fixed
* CopyFile returns true/false if request sent + test
* DeleteFile returns true/false if request sent + test
* getChildrenRoles by checking inheritance
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