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

Implement data aquisition #60

Merged
merged 15 commits into from
Oct 4, 2022
Merged

Implement data aquisition #60

merged 15 commits into from
Oct 4, 2022

Conversation

nstephenh
Copy link
Contributor

@nstephenh nstephenh commented Sep 28, 2022

Implementing #25

  • Load list from gallery
  • Create workspace from selected cat
  • Create workspace from playground when it's selected.

Draft PR as I still have to create the workspace and implement something for playground.

@nstephenh nstephenh marked this pull request as ready for review October 3, 2022 23:15
@nstephenh
Copy link
Contributor Author

Whoops, have to push first :P. I'm used to having a "commit and push" button.

<ProjectReference Include="..\WarHub.ArmouryModel.Concrete.Extensions\WarHub.ArmouryModel.Concrete.Extensions.csproj" />
<ProjectReference Include="..\WarHub.ArmouryModel.EditorServices\WarHub.ArmouryModel.EditorServices.csproj" />
<ProjectReference Include="..\WarHub.ArmouryModel.Extensions\WarHub.ArmouryModel.Extensions.csproj" />
</ItemGroup>

<ItemGroup>
<!-- <ServiceWorker Include="wwwroot\service-worker.js" PublishedContent="wwwroot\service-worker.published.js" /> -->
Copy link
Member

Choose a reason for hiding this comment

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

I don't think these changes are necessary. wwwroot/appsettings.json will automatically be included.

Copy link
Member

Choose a reason for hiding this comment

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

Lines 29-35.

<ProjectReference Include="..\WarHub.ArmouryModel.Concrete.Extensions\WarHub.ArmouryModel.Concrete.Extensions.csproj" />
<ProjectReference Include="..\WarHub.ArmouryModel.EditorServices\WarHub.ArmouryModel.EditorServices.csproj" />
<ProjectReference Include="..\WarHub.ArmouryModel.Extensions\WarHub.ArmouryModel.Extensions.csproj" />
</ItemGroup>

<ItemGroup>
<!-- <ServiceWorker Include="wwwroot\service-worker.js" PublishedContent="wwwroot\service-worker.published.js" /> -->
Copy link
Member

Choose a reason for hiding this comment

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

Lines 29-35.

@nstephenh
Copy link
Contributor Author

We decided to skip creating the workspace from playground for now, until we have a workflow to show the workspace and integrate this with.

@nstephenh nstephenh merged commit ac5b8dc into main Oct 4, 2022
@nstephenh nstephenh deleted the nsh-load-data branch October 4, 2022 20:30
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