Skip to content

[#74340] 17.4 Account settings changes https://community.openproject…#22991

Open
ehassan01 wants to merge 5 commits intorelease/17.4from
documentation/74340-17-4-account-settings-changes
Open

[#74340] 17.4 Account settings changes https://community.openproject…#22991
ehassan01 wants to merge 5 commits intorelease/17.4from
documentation/74340-17-4-account-settings-changes

Conversation

@ehassan01
Copy link
Copy Markdown
Contributor

….org/wp/74340

[#74340] 17.4 Account settings changes

https://community.openproject.org/wp/74340

Ticket

What are you trying to accomplish?

Screenshots

What approach did you choose and why?

Merge checklist

  • Added/updated tests
  • Added/updated documentation in Lookbook (patterns, previews, etc)
  • Tested major browsers (Chrome, Firefox, Edge, ...)

@github-actions
Copy link
Copy Markdown

Caution

The provided work package version does not match the core version

Details:

Please make sure that:

  • The work package version OR your pull request target branch is correct

@ehassan01
Copy link
Copy Markdown
Contributor Author

Hi @MayaBerd. I didn't change the image for notifications and email settings yet, as it would require multiple screen-snips from my end.

@ehassan01
Copy link
Copy Markdown
Contributor Author

Hi @MayaBerd, I tried to rename the internal link (#notifications-settings) in the table to (#notification-and-email-settings) to better reflect the 17.4 update where notification and email are now merged, but this triggered errors in two other pages where #notifications-settings are linked:

  1. openproject\docs\user-guide\forums\README.md
  2. openproject\docs\user-guide\work-packages\edit-work-package\README.md

So, I restored the naming to the default and the errors are gone.

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.

1 participant