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

WIP: Add analytics dashboard based on Flexmonster #1

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

Conversation

benkrikler
Copy link

No description provided.

</head>
<body>
<!-- TODO: Somehow I need to pass pivot_data url / function the survey_id parameter -->
<div id="pivot-table-container" data-url="{% url 'pivot_data' survey_id=4%}"></div>
Copy link
Author

Choose a reason for hiding this comment

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

@AlexxxH I think this is the line I need to fix now - somehow I need to pass survey_id through and doing it here works, but is obviously hard-coded the survey ID - is there way to access this parameter at this point or some alternative approach I should use?

Copy link
Author

Choose a reason for hiding this comment

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

Ah, I think I might have understood what I was missing (or at least one way to fix this): I should pass survey_id as a parameter to the overall template via this line: https://github.com/THEPortatCERN/Hackathon2020-Pier84/pull/1/files#diff-89918cd6e497bfe3ee6e33b23d2ef640R11

@benkrikler
Copy link
Author

@AlexxxH, Do you need me to rebase this branch on the latest master to pick up the history changes you put in?

@AlexxxH
Copy link
Collaborator

AlexxxH commented Oct 6, 2020

@benkrikler Yes that should be fine, thanks!

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