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

XS✔ ◾ New rule anchoring sales #8492

Closed
wants to merge 7 commits into from
Closed

Conversation

Tanya-Leahy
Copy link
Member

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

Email from Adam - "Jeff Bezos: Truth is uncomfortable | Lex Fridman Podcast Clips"

  1. What was changed?

Added a new rule for Anchoring in Sales and put it in a category

  1. Did you do pair or mob programming (list names)?

Worked with @leahy268

Copy link
Contributor

github-actions bot commented May 3, 2024

PR Metrics

Thanks for keeping your pull request small.
Thanks for adding tests.

Lines
Product Code -
Test Code -
Subtotal -
Ignored Code 50
Total 50

Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs!

Copy link
Contributor

github-actions bot commented May 3, 2024

Hi @Tanya-Leahy,

Rule: ../../rules/anchoring-sales-strategy/rule.md

Issues:

  • must have required property 'guid'
  • must have required property 'created'

Please fix the Frontmatter in your Markdown files.

@github-actions github-actions bot changed the title New rule anchoring sales XS✔ ◾ New rule anchoring sales May 3, 2024
@github-actions github-actions bot added Age: 🥚 - New About 2 hours old Age: 🐣 - Young About 4 hours old and removed Age: 🥚 - New About 2 hours old labels May 3, 2024
Copy link
Contributor

github-actions bot commented May 3, 2024

Hi @Tanya-Leahy,

Rule: ../../rules/anchoring-sales-strategy/rule.md

Issues:

  • must have required property 'guid'
  • must have required property 'created'

Please fix the Frontmatter in your Markdown files.

@github-actions github-actions bot added Age: 🐥 - Adolescent About 8 hours old and removed Age: 🐣 - Young About 4 hours old labels May 3, 2024
Copy link
Member

@leahy268 leahy268 left a comment

Choose a reason for hiding this comment

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

It needs a guid and creation time

Co-authored-by: Warwick Leahy [SSW] <58454534+leahy268@users.noreply.github.com>
@github-actions github-actions bot added Age: 🐤 - Mature About 16 hours old and removed Age: 🐥 - Adolescent About 8 hours old labels May 3, 2024
rules/anchoring-sales-strategy/rule.md Outdated Show resolved Hide resolved
rules/anchoring-sales-strategy/rule.md Outdated Show resolved Hide resolved
rules/anchoring-sales-strategy/rule.md Outdated Show resolved Hide resolved
@github-actions github-actions bot added Age: 🐓 - Old About 32 hours old 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: 🐤 - Mature About 16 hours old Age: 🐓 - Old About 32 hours old labels May 3, 2024
Copy link
Contributor

github-actions bot commented May 4, 2024

Howzit @Tanya-Leahy,

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!

@github-actions github-actions bot added Age: 🦖 - Extinct About 128 hours old and removed Age: 🍗 - Ancient About 64 hours old labels May 5, 2024
@Tanya-Leahy Tanya-Leahy marked this pull request as draft May 12, 2024 22:37
@tiagov8
Copy link
Member

tiagov8 commented May 14, 2024

FYI - as per @Tanya-Leahy ,

This PR is o hold for @tiagov8 to check and compare to https://www.ssw.com.au/rules/recognize-anchoring-effects/

@Tanya-Leahy
Copy link
Member Author

cc: @adamcogan @miichaelsmedley

As per conversation with @uly1
This PR was initially created based on the email - Jeff Bezos: Truth is uncomfortable | Lex Fridman Podcast Clips.
However, we at SSW do not recommend using anchoring. Reasons are on this rule:
https://www.ssw.com.au/rules/recognize-anchoring-effects/

As per conversation with @tiagov8 we agreed to close this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Age: 🦖 - Extinct About 128 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.

None yet

3 participants