Skip to content

Conversation

@Mary-Clb
Copy link
Contributor

@Mary-Clb Mary-Clb commented Dec 5, 2025

  • I have read the CONTRIBUTING document.
  • I have performed a self-review of my code.
  • I have added tests that prove my fix is effective or that my feature works.
  • This change requires a documentation update.

Description

It fixes !40258
The project creation via templates workflow had a confusing UX where users could add elements as tasks and teams to templates, thinking they were creating actual projects.

The issue was resolved by implementing 2 elements :

  • Bootstrap modal dialog that appears when users click on any project template for creation : the users must explicitly confirm their intention before project creation proceeds.
  • Warning banners in the project form when users are in template context, explaining they have to first register the project before adding any elements, as otherwise they will only be created on the template.

##Screenshots

project-creation-modal warning-empty-template warning-not-empty-template

@Mary-Clb Mary-Clb added the bug label Dec 5, 2025
<div>
<strong>{{ __('Warning') }} :</strong>
{% if item.getID() == -1 %}
Vous créez un projet depuis un template vide. Remplissez les informations du projet et cliquez sur "Ajouter" pour le créer.
Copy link
Contributor

Choose a reason for hiding this comment

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

All strings should be translatable, and redacted in english.

Copy link
Contributor

@trasher trasher left a comment

Choose a reason for hiding this comment

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

Not sure on the feature itself;, but that's the same for all "templates"; not only for projects.
While I'd agree the UI should distinct a bit; displaying huge warnings is probably counter productive.

@Mary-Clb Mary-Clb marked this pull request as draft December 5, 2025 15:18
@Mary-Clb
Copy link
Contributor Author

Mary-Clb commented Dec 9, 2025

Replaced by #22293

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.

2 participants