Skip to content

DO NOT MERGE: Adding flows and amending UI to handle different calm types to view#1

Open
LeighFinegold wants to merge 1 commit intojpgough:mainfrom
LeighFinegold:calm-hub-flows
Open

DO NOT MERGE: Adding flows and amending UI to handle different calm types to view#1
LeighFinegold wants to merge 1 commit intojpgough:mainfrom
LeighFinegold:calm-hub-flows

Conversation

@LeighFinegold
Copy link

@LeighFinegold LeighFinegold commented Dec 1, 2024

Summarising evening play with calm-hub

  • Added a little README.md for running up mongo locally.
  • Fairly easy to extend to host flows.
  • Added traderx in initmongo script
  • Change UI so that it supports all forms of calm-types

CalmHubUIEnhancement

  • Main feedback to @jpgough is around using integer counters to represent whole store globally or not. (The counter left in DB is not correct given init-mongo script)
  • Whether any plan to validate the json document been created along the schema
  • If there are no patterns or flows for a given namespace, the backend gives NullPointer but has no impact to UI
  • What is the difference between the uniqueId in the calm doc vs the unique id given by calmhub?

@LeighFinegold LeighFinegold changed the title Adding flows and amending UI to handle different calm types to view DO NOTE MERGE: Adding flows and amending UI to handle different calm types to view Dec 1, 2024

## Working with the project

### Running mongo
Copy link
Author

Choose a reason for hiding this comment

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

Not saying one needs to use mongo:4.0.

I have an old mac and 4.2 didn't work :D

if (db.counters.countDocuments({ _id: "flowStoreCounter" }) === 1) {
db.counters.insertOne({
_id: "flowStoreCounter",
sequence_value: 1
Copy link
Author

Choose a reason for hiding this comment

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

I think if I was to play around a bit more with the swagger api, I probably will not be able to add documents because this init mongo script adds more documents then the initial sequence number.

@LeighFinegold LeighFinegold changed the title DO NOTE MERGE: Adding flows and amending UI to handle different calm types to view DO NOT MERGE: Adding flows and amending UI to handle different calm types to view Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant