Skip to content
This repository was archived by the owner on Sep 11, 2023. It is now read-only.

add initial Readme +diagram #361

Merged
merged 3 commits into from
Nov 11, 2021
Merged

add initial Readme +diagram #361

merged 3 commits into from
Nov 11, 2021

Conversation

peterdudfield
Copy link
Contributor

Pull Request

Description

  • add diagram
  • add simple readme

Screenshot 2021-11-10 at 10 26 43

Fixes #324

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@peterdudfield peterdudfield marked this pull request as ready for review November 10, 2021 15:05
@peterdudfield
Copy link
Contributor Author

I need to remove the ~~~~ red line in the image

@peterdudfield peterdudfield self-assigned this Nov 11, 2021
JackKelly
JackKelly approved these changes Nov 11, 2021
View reviewed changes
Copy link
Member

@JackKelly JackKelly left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks loads!

@peterdudfield peterdudfield merged commit 3857645 into main Nov 11, 2021
@peterdudfield peterdudfield deleted the issue/324-diagram branch November 11, 2021 13:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@JackKelly JackKelly JackKelly approved these changes

Assignees

@peterdudfield peterdudfield

Labels
None yet
Projects
No open projects
Nowcasting
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Document new architecture (including a diagram)
2 participants
@peterdudfield @JackKelly