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

fix: [DHIS2-15633] Change comment to note #3549

Merged
merged 25 commits into from
Jul 15, 2024

Conversation

henrikmv
Copy link
Contributor

@henrikmv henrikmv commented Mar 1, 2024

DHIS2-15633

  • Changed comment to note in all the displayed texts across the pages.
  • Changed comment to note in code

@henrikmv henrikmv changed the title fix: [DHIS2-15633] Change ‘comment’ to 'note' fix: [DHIS2-15633] Change comment to note Apr 19, 2024
@henrikmv henrikmv marked this pull request as ready for review April 19, 2024 16:10
@henrikmv henrikmv requested a review from a team as a code owner April 19, 2024 16:10
@henrikmv henrikmv requested a review from JoakimSM April 19, 2024 16:11
Copy link
Contributor

@simonadomnisoru simonadomnisoru left a comment

Choose a reason for hiding this comment

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

Hey @henrikmv, it looks good. I just have a suggestion if you also change "comment" to "note" in the docs too. Thanks!

@henrikmv
Copy link
Contributor Author

henrikmv commented May 25, 2024

Thanks for the reminder about the docs @simonadomnisoru. I did an update for the documentation in PR #3649

Copy link

github-actions bot commented May 27, 2024

@henrikmv henrikmv requested review from a team as code owners May 27, 2024 16:33
Copy link

@geethaalwan geethaalwan left a comment

Choose a reason for hiding this comment

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

Tested successfully on 2.42,2.41.1,2.40.4,2.39.6,2.38.7 versions

@henrikmv
Copy link
Contributor Author

After discussions with @JoakimSM, I have also changed from comment to note in code.

Copy link
Contributor

@simonadomnisoru simonadomnisoru left a comment

Choose a reason for hiding this comment

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

Hi @henrikmv,
Good job 🥳 ! I just added a small comment. In addition to this, I was wondering if it would make sense to run this ticket through some QA regression testing again (given that the change in the codebase is quite large), WDYT?
Thanks!

@linadhis2
Copy link

LGTM from QA perspective and recheck

@henrikmv henrikmv merged commit 56ea017 into master Jul 15, 2024
119 checks passed
@henrikmv henrikmv deleted the hv/fix/DHIS2-15633_ChangeCommentToNote branch July 15, 2024 14:12
dhis2-bot added a commit that referenced this pull request Jul 15, 2024
## [100.70.11](v100.70.10...v100.70.11) (2024-07-15)

### Bug Fixes

* [DHIS2-15633] Change comment to note ([#3549](#3549)) ([56ea017](56ea017))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 100.70.11 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

6 participants