-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat: introduce Opportunity creation for Broken Backlinks #475
Conversation
This PR will trigger a minor release when merged. |
# Conflicts: # package-lock.json # package.json
@@ -216,3 +216,50 @@ export default new AuditBuilder() | |||
.build(); | |||
|
|||
``` | |||
|
|||
### How to add Opportunities and Suggestions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
# Conflicts: # package-lock.json
const newDataKeys = new Set(newData.map(buildKey)); | ||
const existingSuggestions = await opportunity.getSuggestions(); | ||
|
||
// Remove outdated suggestions |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the existing suggestion was entered via back-office-ui as the source, we should NOT delete it, irrespective of whether the same suggestion exists in the new audit or not
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will handles this as a next step. This will require an additional property in the data model
# [1.36.0](v1.35.0...v1.36.0) (2024-11-29) ### Features * introduce Opportunity creation for Broken Backlinks ([#475](#475)) ([78ce9a3](78ce9a3))
🎉 This PR is included in version 1.36.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
No description provided.