Skip to content

initial pull request#254

Open
Amy-O wants to merge 3 commits intomakersacademy:mainfrom
Amy-O:main
Open

initial pull request#254
Amy-O wants to merge 3 commits intomakersacademy:mainfrom
Amy-O:main

Conversation

@Amy-O
Copy link

@Amy-O Amy-O commented Jul 8, 2022

No description provided.

<input type="text" name="name">
<input type="submit" value="Submit your name">
</form>

Choose a reason for hiding this comment

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

There are 3 .ERB files. Wondering if it is recommended to have just 1 .ERB file as has been the practice in the previous excercises

run! if app_file == $0

end

Choose a reason for hiding this comment

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

Code is concise and readable!

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.

2 participants