Skip to content
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

Scaffolding (on 1.0 project) a view with model and without dbContext fails for Delete, Details and List templates #425

Closed
prafullbhosale opened this issue Mar 23, 2017 · 8 comments

Comments

@prafullbhosale
Copy link
Contributor

prafullbhosale commented Mar 23, 2017

Repro Steps:

  1. Create a new Asp.Net Core project with Individual auth (1.0)
  2. Add a model class
public class Car 
{
   public int ID { get; set;}
   public class Name { get; set;}
}
  1. Right click Views folder -> Add View

Scaffolding fails with below error
image

@prafullbhosale prafullbhosale added this to the 1.0.1 milestone Mar 23, 2017
@prafullbhosale prafullbhosale self-assigned this Mar 23, 2017
@prafullbhosale prafullbhosale changed the title Scaffolding a view with model and without dbContext fails for Delete, Details and List templates Scaffolding (on 1.0 project) a view with model and without dbContext fails for Delete, Details and List templates Mar 23, 2017
@Eilon
Copy link
Member

Eilon commented Apr 11, 2017

This patch fix is approved. Please follow the normal code review / pull request process and make sure you make the change in the correct branch.

@prafullbhosale
Copy link
Contributor Author

I am assuming the other issue for 1.1 (#427) is also approved.

@Eilon
Copy link
Member

Eilon commented Apr 11, 2017

@prafullbhosale I will approve it shortly; just waiting for the build system to be up and running for the patch.

@Eilon
Copy link
Member

Eilon commented Apr 18, 2017

@prafullbhosale / @mlorbetske - do you guys have a status update on this bug? We want to get the patches done ASAP.

@prafullbhosale
Copy link
Contributor Author

The branches are setup and included in the Coherence-Patch builds.
Only remaining part is to bump up the versions for latest packages. Have those on PR #453 and #452

@prafullbhosale
Copy link
Contributor Author

Fixed in rel/tools-1.0.1

@Eilon
Copy link
Member

Eilon commented Apr 19, 2017

Preview builds of this patch fix should be available on the following feeds:

If you have a chance, please try out these preview builds and let us know if you have any feedback!

@prafullbhosale
Copy link
Contributor Author

Tested the scaffolding packages from the feed, the fix looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants