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

Copy job to another project #2546

Merged
merged 5 commits into from
Jun 20, 2017
Merged

Conversation

jtobard
Copy link
Contributor

@jtobard jtobard commented Jun 5, 2017

GUI enhancement to allow copy a job to another project, only if the user is authorized to create in the other project, uses the same copy method only overwriting the project for the selected one

New item in dropdown menu "Duplicate this job to other Project":
1_dropdown_menu

Popup asking to choose the project in the autocomplete list:
2_select_project

If the user can't create in any other project, the user don't get the option on the menu:
3_dropdown_menu_wo_perm

This is how it looks if the user can create in other project but only can read on the actual:
4_dropdown_other_not_this

If the user tries to duplicate to a project he can't see:
5_error_cant_see_pr

If the user tries to duplicate to a project he can see but cant create:
6_error_noauth_create

@gschueler
Copy link
Member

issues:

  1. does not work from the job show page. results in 404 /scheduledExecution/copyanother
  2. should use a Select not a text field for project select

@gschueler
Copy link
Member

also has the same problem #1 from the execution show page

@gschueler gschueler self-requested a review June 14, 2017 22:22
@jtobard
Copy link
Contributor Author

jtobard commented Jun 15, 2017

problems fixed in last commit

Copy link
Member

@gschueler gschueler left a comment

Choose a reason for hiding this comment

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

👍

@gschueler gschueler merged commit bbcf13b into rundeck:master Jun 20, 2017
@jtobard jtobard deleted the dev/copy-job-project branch July 19, 2017 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants