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.

Sign up for GitHub

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

Jump to bottom

Close (sub)project warning message #1880

Merged

galethil merged 2 commits into main from 1871-warning-message

Jul 4, 2024

Collaborator

galethil commented Jun 17, 2024 •

edited

Loading

Checklist

I have followed (at least) the PR section of the contributing guide.
I fixed all necessary PR warnings
The commit history is clean
The E2E tests are passing
If possible, the issue has been divided into more subtasks
I did a self review before requesting a review from another team member

Description

How to test

Create new project and subproject
Close subproject
Close project
In both cases you should see the warning in all languages

Closes #1871


          ui:Close (sub)project warning message

4d66e90

github-actions bot commented Jun 17, 2024 •

edited

Loading

	Warnings
⚠️	No CHANGELOG added.
⚠️	There were changes in the frontend, but no E2E-test was added or modified!

Generated by 🚫 dangerJS against b77df20

openkfwCI commented Jun 17, 2024 •

edited

Loading

	Notes	Time
✅	Note for Reviewer: E2E tests on pipeline 59772 on remote server succeeded	Wed, 03 Jul 2024 09:26:21 +0000

Generated by E2E-Test

SamuelPull approved these changes

View reviewed changes


          Merge branch 'main' into 1871-warning-message

b77df20

galethil merged commit 1c3f645 into main

30 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet