Skip to content

Migrate GProjectDialog to script setup#1934

Merged
petersutter merged 2 commits into
masterfrom
enh/project-dialog-script-setup
Jun 26, 2024
Merged

Migrate GProjectDialog to script setup#1934
petersutter merged 2 commits into
masterfrom
enh/project-dialog-script-setup

Conversation

@petersutter
Copy link
Copy Markdown
Member

@petersutter petersutter commented Jun 21, 2024

What this PR does / why we need it:
Migrate GProjectDialog to script setup.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
Use Hide whitespace for review.

Release note:


@ghost ghost added the needs/review Needs review label Jun 21, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jun 21, 2024
@ghost ghost added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. needs/second-opinion Needs second review by someone else labels Jun 21, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 added needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jun 21, 2024
Copy link
Copy Markdown
Member

@grolu grolu left a comment

Choose a reason for hiding this comment

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

/lgtm

@ghost ghost added reviewed/lgtm Has approval for merging and removed needs/review Needs review needs/second-opinion Needs second review by someone else labels Jun 24, 2024
@gardener-robot-ci-1 gardener-robot-ci-1 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jun 24, 2024
@petersutter petersutter force-pushed the enh/project-dialog-script-setup branch from 027cf0a to aef990d Compare June 25, 2024 11:58
@ghost ghost added needs/second-opinion Needs second review by someone else and removed reviewed/lgtm Has approval for merging reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Jun 25, 2024
Comment thread frontend/src/components/dialogs/GProjectDialog.vue Outdated
Comment thread frontend/src/components/dialogs/GProjectDialog.vue Outdated
Comment thread frontend/src/components/dialogs/GProjectDialog.vue Outdated
@petersutter petersutter requested a review from holgerkoser June 25, 2024 15:50
@gardener-robot-ci-2 gardener-robot-ci-2 added the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jun 25, 2024
Copy link
Copy Markdown
Member

@holgerkoser holgerkoser left a comment

Choose a reason for hiding this comment

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

/lgtm

@ghost ghost added reviewed/lgtm Has approval for merging and removed needs/second-opinion Needs second review by someone else labels Jun 25, 2024
@gardener-robot-ci-2 gardener-robot-ci-2 removed the reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) label Jun 25, 2024
@petersutter petersutter merged commit 58bf69b into master Jun 26, 2024
@petersutter petersutter deleted the enh/project-dialog-script-setup branch June 26, 2024 08:44
@ghost ghost added the status/closed Issue is closed (either delivered or triaged) label Jun 26, 2024
holgerkoser added a commit that referenced this pull request Jun 26, 2024
* master:
  Migrate GProjectDialog to script setup (#1934)

# Conflicts:
#	frontend/src/components/dialogs/GProjectDialog.vue
holgerkoser added a commit that referenced this pull request Jun 26, 2024
* master: (169 commits)
  revert #1933 (#1937)
  Migrate GProjectDialog to script setup (#1934)
  Update connect-kubectl.md
  cancel on esc (#1933)
  fix project menu after vuetify update (#1929)
  Update dependency vuetify to v3.6.10 (#1927)
  Bump braces from 3.0.2 to 3.0.3 (#1922)
  Update dependency vite to v5.3.1 (#1920)
  Update dependency @braintree/sanitize-url to v7.0.3 (#1925)
  Update dependency vue to v3.4.29 (#1921)
  Update dependency jose to v5.4.1 (#1924)
  Update dependency sass to v1.77.6 (#1917)
  Update dependency vuetify to v3.6.9 (#1916)
  Update dependency @vueuse/router to v10.11.0 (#1914)
  Update dependency @vueuse/integrations to v10.11.0 (#1913)
  Update actions/checkout action to v4.1.7 (#1918)
  Update dependency @vueuse/core to v10.11.0 (#1912)
  Adapt the debug and test scripts for `vscode-jest-runner` to be compatible with Vitest. (#1923)
  Update dependency vue-router to v4.3.3 (#1907)
  Update Yarn to v4.3.0 (#1910)
  ...

# Conflicts:
#	.pnp.cjs
#	backend/lib/app.js
#	backend/lib/config/gardener.js
#	backend/lib/security/index.js
#	backend/lib/services/shoots.js
#	frontend/package.json
#	frontend/vite.config.js
#	yarn.lock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) reviewed/lgtm Has approval for merging size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. status/closed Issue is closed (either delivered or triaged)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants