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

Add button in web-ui to trigger the deployment event #1415

Merged
merged 11 commits into from
Dec 1, 2022

Conversation

smainz
Copy link
Contributor

@smainz smainz commented Nov 11, 2022

This PR adds a button to the web ui to trigger the deployment event on the current pipeline.

closes #1412

The button is only active, if the current pipeline is in the state success.

I would like to get some feedback if the path I ave taken to implement this was the correct one (my first ever contact with Vue):

  • add deployPipeline( ... ) to WoodpeckerClient similair to what is done in client.go
  • Copy the ManualPipelinePopup.vue and add a param id for the current pipeline id, change the text ids
  • Add a button into PipelineWrapper to open the new popup
  • Add texts for locales en and de

Is there anything I have forgotten to change?

@qwerty287 qwerty287 changed the title Add button in web-ui to trigger the deployment event (fixes #1412) Add button in web-ui to trigger the deployment event Nov 13, 2022
@6543 6543 added ui frontend related enhancement improve existing features labels Nov 13, 2022
@smainz smainz requested a review from 6543 November 14, 2022 14:32
@anbraten anbraten added this to the 1.0.0 milestone Nov 27, 2022
@smainz smainz marked this pull request as draft November 28, 2022 08:38
@smainz smainz marked this pull request as ready for review November 29, 2022 10:33
@smainz
Copy link
Contributor Author

smainz commented Nov 29, 2022

I improved my Vue skills and this PR is running here in production since yesterday, so it shuld work.

Thanks to the reviewers.

@lafriks
Copy link
Contributor

lafriks commented Nov 30, 2022

screenshot would be nice to add to PR description

@smainz
Copy link
Contributor Author

smainz commented Nov 30, 2022

screenshot would be nice to add to PR description

Screenshot (Mock) is in #1412 or here (actual, german)

The deploy button:
vivaldi_lsSby2P5bF

The window to enter environment params and start deploymnet:
vivaldi_MfjzNT9mWP

Co-authored-by: Lauris BH <lauris@nix.lv>
@anbraten anbraten merged commit cfb2882 into woodpecker-ci:master Dec 1, 2022
@smainz smainz deleted the deploy-button branch December 4, 2022 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement improve existing features ui frontend related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deploy Button in the Web-UI
5 participants