-
Notifications
You must be signed in to change notification settings - Fork 17
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
Feature: Add PR view #21
Comments
I'll take this. |
This one is closed, right? PR #24 closes this. |
@Layla-P had review feedback. I implemented some but had some questions about other points that were raised in that original PR. |
Hello - If you add the text: "Fix #21 " to the relevant PR description it will add it to that issues discussion and close the issue when complete Looks like it already got merged as well. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
** Description: **
Use the existing add.cshtml and add method on the Homecontroller to log a PR
** Steps: **
Fields to capture
We have a DTO prepared for this:
https://github.com/Layla-P/HacktoberfestProject/blob/dev/HacktoberfestProject.Web/Models/DTOs/User.cs
The username will be captured from the HttpContext so no need to have a field for it.
In the Add Method on the Controller, we will use the TableService to save the new PR
The text was updated successfully, but these errors were encountered: