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 an enterprise banner on top of automatic subject configuration #17506

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

NobodysNightmare
Copy link
Contributor

@NobodysNightmare NobodysNightmare commented Dec 19, 2024

We are making sure that users can revert back to manual subjects at any time and even keep using the feature if it's still enabled after their enterprise token expired. They will merely not be able to set new types to automatic and have to live with an enterprise banner being shown to them whenever they change the subject config.

Ticket

https://community.openproject.org/projects/document-workflows-stream/work_packages/59929

What are you trying to accomplish?

Show users that this feature is enterprise-exclusive, while retaining their ability to use the feature (or stop using it) once their EE license has expired

Screenshots

image

What approach did you choose and why?

Our new standard component for EE banners, because standardization is great!

Merge checklist

  • Added/updated tests
  • Tested major browsers (Chrome, Firefox, Edge, ...)

@NobodysNightmare NobodysNightmare marked this pull request as draft December 19, 2024 16:03
@@ -251,6 +251,9 @@ def static_links
href: "https://www.openproject.org/docs/user-guide/time-and-costs/progress-tracking/"
},
enterprise_docs: {
automatic_subject_generation: {
href: "https://www.openproject.org/docs/todo/do-not-merge"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can we already know/guess the correct URL this will have in the future? Otherwise we'll need to remember to revisit this link and fix it before public release.

Copy link
Member

Choose a reason for hiding this comment

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

This needs to be discussed with Maya. Maybe she can predict it, if not, we need to put a placeholder.

Copy link
Contributor Author

@NobodysNightmare NobodysNightmare Dec 20, 2024

Choose a reason for hiding this comment

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

Aligned with @MayaBerd and added a small placeholder into the docs already and pointed to the correct place.

Note: We should now not merge this branch before todays feature freeze (so that the docs are not updated before 15.3).

(Alternatively if we need to merge this today because of dependencies or what not --> just remove the text from the docs and we are safe as well)

@NobodysNightmare NobodysNightmare force-pushed the subject-enterprise-banner branch from 7f9bb0f to ade9446 Compare December 19, 2024 16:10
@NobodysNightmare
Copy link
Contributor Author

@ulferts Not sure, but this might be the first time that we use the new banner component 🎉

As can be seen in the screenshot above, there should be additional space between the banner and the form.

Do you think this is something that I should fix on a "this usage only" basis (e.g. by figuring out how to use system_arguments or if all else fails adding a <br> tag 😉 ) or should we rather introduce a default bottom-margin to the banner?

@NobodysNightmare
Copy link
Contributor Author

I talked to @HDinger in your absence. She expressed a preference in leaving margins/paddings out of the component and defining this on a case-by-case basis

@NobodysNightmare NobodysNightmare force-pushed the subject-enterprise-banner branch 2 times, most recently from f3b4acb to 667902a Compare December 20, 2024 09:37
@NobodysNightmare NobodysNightmare marked this pull request as ready for review December 20, 2024 09:37
@NobodysNightmare NobodysNightmare requested a review from a team December 20, 2024 09:38
Adding a test selector that can be used to detect the
presence of the enterprise banner in other specs and
improving the docs from non-default values.
@NobodysNightmare NobodysNightmare force-pushed the subject-enterprise-banner branch from 667902a to 89c59d9 Compare December 20, 2024 09:52
We are making sure that users can revert back to manual subjects
at any time and even keep using the feature if it's still enabled
after their enterprise token expired. They will merely not be able
to set new types to automatic and have to live with an enterprise
banner being shown to them whenever they change the subject config.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants