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

Updating starter kit #65

Open
wants to merge 5 commits into
base: dev-v8
Choose a base branch
from
Open

Updating starter kit #65

wants to merge 5 commits into from

Conversation

crgrieve
Copy link

@crgrieve crgrieve commented Oct 1, 2019

Adding more MVPs to the "people" content to make it a more representative view of the inclusive community.

We want new users to Umbraco to see that it's a community they too can get involved in. Since the starter kit is one of the first things they'll see, let's make sure it give us a chance to show how welcoming the community is.

fixing duplicate id error and changing order
@emmaburstow
Copy link

Hey @crgrieve,

Thanks for the Pr and congrats on your first to this repo! I love this idea. Good work!

Em

@crgrieve
Copy link
Author

hello,

@nul800sebastiaan , as discussed at DUUGFest, I asked all the people I added to starter kit. They have all confirmed they are happy to be included :)

Thanks,
Carole

@nul800sebastiaan
Copy link
Member

Thanks @crgrieve - we've done some experimenting with this and it almost works, I found one small problem when installing the SK in a fresh empty website and that is that the media pickers are not connecting the correct media items for the new people you put in there:

image

Could you try to have a look into that? Don't spend a crazy amount of time on it though, the setup of this package is not super easy to figure out and I don't want to waste your time.

Do you, by any chance still have the SQL CE database that was updated when you created these new nodes in Umbraco? It would be really nice if the next person who clones this repo gets the complete set of people in there as well.

@crgrieve
Copy link
Author

Hello,

I tested by building it as a package and installing in a fresh project at the time. I will test again and see if I also get the media item issue.

I should have the .sdf file, it's on a different PC than I am on just now so will check later and get back to you.

Thanks,
Carole

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