Skip to content

Create 4_twitterFunctions.sol - #44

Open
rifqij wants to merge 1 commit into
Aniket-Engg:masterfrom
rifqij:patch-1
Open

rifqij wants to merge 1 commit into
Aniket-Engg:masterfrom
rifqij:patch-1

Conversation

@rifqij

@rifqij rifqij commented Oct 28, 2022

Copy link
Copy Markdown

Smart contract that mimics the functionalities of Twitter

Smart contract that mimics the functionalities of Twitter
@rifqij rifqij closed this Oct 28, 2022
@rifqij rifqij reopened this Oct 28, 2022

@Aniket-Engg Aniket-Engg left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Consider adding events too.

return userTweets;
}

modifier accountExists(address _user) {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

As per solidity style guide here: remix-project-org/remix-project#4034 , Modifiers should come before the functions

This branch has not been deployed

No deployments
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.

4 participants