Skip to content

Define Interactions#11

Open
chris-chris wants to merge 2 commits into
ubiquibot:mainfrom
chris-chris:feat/define-interaction
Open

Define Interactions#11
chris-chris wants to merge 2 commits into
ubiquibot:mainfrom
chris-chris:feat/define-interaction

Conversation

@chris-chris
Copy link
Copy Markdown
Contributor

Resolves #4

For each component, define how it interacts with others. This includes:

Data Flow: How data is passed between components.
Dependencies: Components that rely on others to function.
Event Triggers: Events in one component that trigger actions in another.

@chris-chris
Copy link
Copy Markdown
Contributor Author

@pavlovcik @gentlementlegen
Only ubiquity/ubiquibot-kerneluses cloudflare KV, right?
I searched the cloudflare kv keyword in the organization.

@chris-chris
Copy link
Copy Markdown
Contributor Author

@snibug plz check

@chris-chris

This comment was marked as outdated.

@chris-chris
Copy link
Copy Markdown
Contributor Author

@pavlovcik
I organized github repos into several groups.
I connected each service to DB and API.
Feel free to give me some feedback

@chris-chris
Copy link
Copy Markdown
Contributor Author

@chris-chris chris-chris mentioned this pull request Apr 6, 2024
@chris-chris

This comment was marked as outdated.

@chris-chris chris-chris mentioned this pull request Apr 6, 2024
@chris-chris
Copy link
Copy Markdown
Contributor Author

update
UbiquityDAO Interaction drawio (6)

@chris-chris
Copy link
Copy Markdown
Contributor Author

Please give me some feedback so that I can proceed to the next task.

@gentlementlegen
Copy link
Copy Markdown
Member

gentlementlegen commented Apr 8, 2024

@chris-chris Thank you for the graph, very helpful to see things clearly. If you can update audit.ubq.fi with a proper description, and also link Supabase to comment-incentives, permit-generation, conversation-rewards and comment-incentives as it is used there to retrieve user's wallets.

Also what format do you intend to share this diagram? Maybe the URL you shared should also be included in the document.

@0x4007
Copy link
Copy Markdown
Contributor

0x4007 commented Apr 8, 2024

Also what format do you intend to share this diagram? Maybe the URL you shared should also be included in the document.

Hey Chris I think you went too far ahead because diagramming should come at a later step

The reason for this is because it is extra work for you to redo drawing parts if they are incorrect. For example, you have pay.ubq.fi under "token solidity" but really our only custom smart contracts are for @ubiquity/ubiquity-dollar

In pay.ubq.fi technically there is a permit generation tool (using Uniswap's contract) but its for debugging/development purposes only. The UI simply parses JSON encoded permit data, which is meant to be generated elsewhere (by the bot, posted on GitHub)

Anyways there is plenty of more small adjustments we still will need to make before diagramming I think!

Given that you already have something diagramming related, it could also be really helpful if we were able to directly edit it with our adjustments!


I can see you're working on multiple tasks at the same time related to this project. In this case what I've normally seen is that contributors who are interested to be involved will be working on some codebase somewhere in the background in parallel, for example, fixing some problem at the pay.ubq.fi codebase and then pushing forward on this documentation stuff as we are able to provide feedback/input.

@chris-chris
Copy link
Copy Markdown
Contributor Author

chris-chris commented Apr 10, 2024

@0x4007 Thank you for the feedback! 👍 Indeed, organizing the data flow in writing makes sense sequentially. While writing, I got confused about defining the relationships, so I started to draw. I drew the diagram quickly, not just for the output, but also to help myself understand the system. I find that drawing helps me organize my thoughts more effectively.

I'll check the codes in @ubiquity/ubiquity-dollar and pay.ubq.fi.

@0x4007
Copy link
Copy Markdown
Contributor

0x4007 commented Apr 10, 2024

Also what format do you intend to share this diagram? Maybe the URL you shared should also be included in the document.

it could also be really helpful if we were able to directly edit it with our adjustments!

Can you share the URL so we can edit it?

@chris-chris
Copy link
Copy Markdown
Contributor Author

Here's draw.io diagram

https://drive.google.com/file/d/1PoFGrKjxVKtjhDB8fbV0BKPfirvgIhUt/view?usp=sharing

@0x4007 @gentlementlegen
Here it is 😀

@0x4007
Copy link
Copy Markdown
Contributor

0x4007 commented Apr 10, 2024

Here's draw.io diagram
https://drive.google.com/file/d/1PoFGrKjxVKtjhDB8fbV0BKPfirvgIhUt/view?usp=sharing

@0x4007 @gentlementlegen Here it is 😀

Looks like read only access, can't make adjustments.

@gentlementlegen
Copy link
Copy Markdown
Member

Same here, it is on read-only access. However it makes sense that not anyone can modify this document. Maybe it should be moved in the Ubiquity Google Drive to restrict access to collaborators.

@chris-chris
Copy link
Copy Markdown
Contributor Author

@gentlementlegen @0x4007
https://drive.google.com/file/d/1PoFGrKjxVKtjhDB8fbV0BKPfirvgIhUt/view?usp=sharing
I changed it to an editable link.
How can I upload the file to Ubiquity Google Drive?

@0x4007
Copy link
Copy Markdown
Contributor

0x4007 commented Apr 13, 2024

@gentlementlegen @0x4007

https://drive.google.com/file/d/1PoFGrKjxVKtjhDB8fbV0BKPfirvgIhUt/view?usp=sharing

I changed it to an editable link.

How can I upload the file to Ubiquity Google Drive?

Telegram message me your Google drive email and I'll add you to the folder!

@rndquu
Copy link
Copy Markdown

rndquu commented Jun 4, 2024

@chris-chris Is it possible to edit this diagram https://drive.google.com/file/d/1PoFGrKjxVKtjhDB8fbV0BKPfirvgIhUt/view ?

I tried to open that document via draw.io google app but got this error:
Screenshot 2024-06-04 at 18 27 48

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.

Define Interactions

4 participants