-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fyyur Project: Add missing fields, fix broken pages, fix method that throws error, add edit buttons #117
Merged
Commits on Feb 21, 2021
-
Fix broken form. Change text on button to Artist, not venue. Correct facebook_link label.
Configuration menu - View commit details
-
Copy full SHA for 3f5d68c - Browse repository at this point
Copy the full SHA 3f5d68cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 7c8008f - Browse repository at this point
Copy the full SHA 7c8008fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 0f888c6 - Browse repository at this point
Copy the full SHA 0f888c6View commit details -
Configuration menu - View commit details
-
Copy full SHA for f211510 - Browse repository at this point
Copy the full SHA f211510View commit details -
Configuration menu - View commit details
-
Copy full SHA for a8da1d8 - Browse repository at this point
Copy the full SHA a8da1d8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5a7a012 - Browse repository at this point
Copy the full SHA 5a7a012View commit details -
Add edit button to show_artist.html
Add missing button connected to the artist/<artist.id>/edit endpoint. "Todo" instructions say to complete this endpoint, although the front end parts that are supposed to be complete do not use it.
Configuration menu - View commit details
-
Copy full SHA for e0a9e82 - Browse repository at this point
Copy the full SHA e0a9e82View commit details -
Add edit button to show_venue.html
Added missing edit button to connect to the /venues/<venue.id>/edit endpoint. The instructions say to complete this, but it is not used by the completed front end. This will make it much easier for users to test the endpoint when they complete it.
Configuration menu - View commit details
-
Copy full SHA for a67dbf5 - Browse repository at this point
Copy the full SHA a67dbf5View commit details -
Added missing "locale='en'" that causes a crash.
Configuration menu - View commit details
-
Copy full SHA for 25ff47d - Browse repository at this point
Copy the full SHA 25ff47dView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.