Skip to content

Update README.md#36

Open
emailgregn wants to merge 2 commits into
RobWunderlich:masterfrom
emailgregn:patch-1
Open

Update README.md#36
emailgregn wants to merge 2 commits into
RobWunderlich:masterfrom
emailgregn:patch-1

Conversation

@emailgregn

Copy link
Copy Markdown

Make usage explicit so casual users don't have to double google.

Make usage explicit so casual users don't have to double google.
@veglar

veglar commented Sep 10, 2015

Copy link
Copy Markdown

Psst...you have misspelled QliView to Qlickview

@emailgregn

Copy link
Copy Markdown
Author

Doh!
fixed

@RobWunderlich

Copy link
Copy Markdown
Owner

Sorry for the delay on picking this up. I like the idea, but disagree with the specific instructions. There is no need to copy the Runtime to the same folder as the qvw. That could be a bad practice as you can wind up with multiple copies of the Runtime. I think it's better to keep one copy.

Also, I think it's important to have people CALL Qvc.Cleanup.

How about:
Quickstart:

  1. Unzip the distribution
  2. Add $(Include=path to unzipped folder\qvc_runtime\qvc.qvs);
    at the top of the QlikView load script.
  3. Call Qvc.* routines as required.
  4. At the end of your script, CALL Qvc.Cleanup.

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.

3 participants