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

CLI bug fixes #2184

Merged
merged 8 commits into from Apr 8, 2014
Merged

CLI bug fixes #2184

merged 8 commits into from Apr 8, 2014

Conversation

joshmoore
Copy link
Member

Various dangling CLI tickets from 5.0:

12024

http://trac.openmicroscopy.org.uk/ome/ticket/12024 to filter inactive users:

$ bin/omero user add -e "test@example.com" emailtest Email Tester sudotest
Please enter password for your new user (emailtest): 
Please re-enter password for your new user (emailtest): 
Added user emailtest (id=802) with password
$ bin/omero user email -i
test@example.com, ...other addresses...
$ bin/omero user email -i --all
test@example.com, ...other addresses...
$ bin/omero group removeuser --name user emailtest
Removed 802 from group 1
$ bin/omero user email -i
...other addresses...
$ bin/omero user email -i --all
test@example.com, ...other addresses...

12002

http://trac.openmicroscopy.org.uk/ome/ticket/12002#comment:5
bin/omero config operations should now only save once and that they do via write-and-rename.

@mtbc
Copy link
Member

mtbc commented Mar 30, 2014

Travis failure.

@manics manics mentioned this pull request Apr 7, 2014
6 tasks
Rather than considering all non-default environments
as failures on read-only, only consider it a failure
when read-only was not *explicitly* requested.
Travis is failing with flake8 errors. Merging this
in so that everything is up-to-date.
@ximenesuk
Copy link
Contributor

Functions as expected, unit tests passes, code looks okay. Good to merge unless this is waiting on anything else.

joshmoore added a commit that referenced this pull request Apr 8, 2014
@joshmoore joshmoore merged commit c451d13 into ome:dev_5_0 Apr 8, 2014
@joshmoore joshmoore deleted the cli-bug-fixes branch April 8, 2014 15:49
@joshmoore
Copy link
Member Author

--rebased-to #2273

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants