Skip to content
This repository has been archived by the owner on May 26, 2020. It is now read-only.

Issue 40 #44

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Issue 40 #44

wants to merge 3 commits into from

Conversation

m-gagne
Copy link
Collaborator

@m-gagne m-gagne commented Jun 14, 2017

Resolves Issue #40

  • Moves samples into ./samples//.html
  • Removes samples that refer to published .js (uses local .js and .css only) for AzSearch.js and AzSearch.css
  • Resolves HTML lint issues with missing closing tags () and scripts outside of

@EvanBoyle
Copy link
Collaborator

EvanBoyle commented Jun 14, 2017 via email

@EvanBoyle
Copy link
Collaborator

Yeah what do you think about versioning the samples. I think it's a benefit that we refer to a specfiic verison in the samples. It makes it easy to download the sample and run it yourself. Otherwise someone has to build the entire project. Having the realestate.html and the realestate.local.html nicely avoids this.

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

Successfully merging this pull request may close these issues.

2 participants