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

Fix circular dep #3067

Closed
wants to merge 3 commits into from
Closed

Fix circular dep #3067

wants to merge 3 commits into from

Conversation

oluoluoxenfree
Copy link
Contributor

@oluoluoxenfree oluoluoxenfree commented Feb 5, 2020 •
edited

Resolves #3054

Overall change: A very high-level summary of easily-reproducible changes that can be understood by non-devs.

Code changes:

  • A bullet point list of key code changes that have been made.
  • When describing code changes, try to communicate how and why you implemented something a specific way, not just what has changed.
  • I have assigned myself to this PR and the corresponding issues
  • Automated jest tests added (for new features) or updated (for existing features)
  • This PR requires manual testing

@oluoluoxenfree oluoluoxenfree self-assigned this Feb 5, 2020
@oluoluoxenfree oluoluoxenfree added the ws-articles Tasks for the WS Articles Team label Feb 5, 2020
@oluoluoxenfree oluoluoxenfree added this to PR in Progress in Simorgh via automation Feb 5, 2020
Simorgh automation moved this from PR in Progress to Done Feb 20, 2020
@sareh sareh deleted the fix-circular-dep branch February 21, 2020 12:30
This was referenced Jul 29, 2020
Merged
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@oluoluoxenfree oluoluoxenfree

Labels
ws-articles Tasks for the WS Articles Team
Projects
No open projects
Simorgh
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Resolve circular dependency
1 participant
@oluoluoxenfree