Skip to content
This repository has been archived by the owner on Feb 13, 2025. It is now read-only.

MWPW-141282 Update UK/AU Blog Tag #31

Merged
merged 2 commits into from
Feb 6, 2024
Merged

MWPW-141282 Update UK/AU Blog Tag #31

merged 2 commits into from
Feb 6, 2024

Conversation

meganthecoder
Copy link
Collaborator

  • Update UK and AU query index urls
  • Change feed value from text to link
  • Fix report saving issue

Resolves: MWPW-141282

Brandon32
Brandon32 reviewed Feb 6, 2024
View reviewed changes
JasonHowellSlavin
JasonHowellSlavin reviewed Feb 6, 2024
View reviewed changes
Copy link
Collaborator

@JasonHowellSlavin JasonHowellSlavin left a comment

Choose a reason for hiding this comment

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

Looks good, I see an update to mocks, did we need to update tests?

@meganthecoder
Copy link
Collaborator Author

Looks good, I see an update to mocks, did we need to update tests?

@JasonHowellSlavin No, the updates to the output files are all that are necessary since the related tests are basically that the mdast created by the code matches the mdast in the output files.

@meganthecoder meganthecoder merged commit ab77332 into adobecom:main Feb 6, 2024
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Brandon32 Brandon32 Brandon32 approved these changes

@JasonHowellSlavin JasonHowellSlavin JasonHowellSlavin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@meganthecoder @Brandon32 @JasonHowellSlavin