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

Update rule.md #9907

Merged
merged 8 commits into from
Feb 20, 2025
Merged

Update rule.md #9907

merged 8 commits into from
Feb 20, 2025

Conversation

ClaraFang-SSW
Copy link
Member

Add instructions about browser language settings and update the screenshots of current operating system

  1. What triggered this change? (PBI link, Email Subject, conversation + reason, etc)

✏️Email: Power BI - TimePro Transfer Payment - Date format wrong
During my transfer payment meeting with Adam, we found out the date format in Power BI was incorrect due to the browser language setting was default as English (UK).

  1. What was changed?

✏️I add an instruction about browser language settings and update the screenshot for operating system

  1. I paired or mob programmed with:

✏️ @ZhangRan8

Add instructions about browser language settings and update the screenshots of current operating system
@github-actions github-actions bot added Age: 🥚 - New About 2 hours old Age: 🐣 - Young About 4 hours old Age: 🐥 - Adolescent About 8 hours old and removed Age: 🥚 - New About 2 hours old Age: 🐣 - Young About 4 hours old labels Feb 18, 2025
@github-actions github-actions bot added Age: 🐤 - Mature About 16 hours old and removed Age: 🐥 - Adolescent About 8 hours old labels Feb 18, 2025
@github-actions github-actions bot added Age: 🐓 - Old About 32 hours old and removed Age: 🐤 - Mature About 16 hours old labels Feb 19, 2025
@JeanThirion JeanThirion self-requested a review February 19, 2025 14:36
@github-actions github-actions bot added Age: 🍗 - Ancient About 64 hours old 🔥 Merge Debt This PR contains merge debt, see https://www.ssw.com.au/rules/merge-debt/ and removed Age: 🐓 - Old About 32 hours old labels Feb 19, 2025
Copy link
Contributor

Howzit @ClaraFang-SSW,

This PR has been here a while.

Did you know you should avoid merge debt?

  1. Please action (e.g. get a review) and merge or close

Thanks!

Hey @ClaraFang-SSW 

Please see all my extra changes to improve this rule

The most important one is the image captions that you did wrongly. 
More info: https://www.ssw.com.au/rules/rule/#6-images
@tiagov8 tiagov8 merged commit a87870a into SSWConsulting:main Feb 20, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Age: 🍗 - Ancient About 64 hours old 🔥 Merge Debt This PR contains merge debt, see https://www.ssw.com.au/rules/merge-debt/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants