acorn icon indicating copy to clipboard operation
acorn copied to clipboard

Update lib.rs

Open adamth0mps0n opened this issue 2 years ago • 1 comments

I have simplified the code by combining functions that appear to be performed the same task.

For example, the create_whoami and inner_create_whoami functions both perform the same task of creating a new profile and linking it to the "agents" path and the agent's pubkey. These two functions have now been combined into one function to reduce redundancy.

Additionally, the use of the get_peers function in both create_whoami and inner_create_whoami have been refactored to a single function that is called by both, rather than duplicating the function call.

The create_imported_profile function also performs similar tasks to the other two functions, so I have taken a chance and combined it with them as well. Haven't tested yet but should work.. give it a try.

adamth0mps0n avatar Jan 28 '23 09:01 adamth0mps0n

Hi @adamth0mps0n I can see why this might be an improvement, but it also appears that some of the other necessary functions like fetch_agents have been removed.

Connoropolous avatar Feb 02 '23 03:02 Connoropolous