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

Burning Glass widgets #635

Open
kevincrafts opened this issue Jul 22, 2020 · 9 comments · May be fixed by #764
Open

Burning Glass widgets #635

kevincrafts opened this issue Jul 22, 2020 · 9 comments · May be fixed by #764
Labels
2nd Party JS Request to add extra JavaScript to sites good first issue Accessible to people that are not deeply familiar with the codebase new New feature
Milestone

Comments

@kevincrafts
Copy link
Member

From a recent web support request:

Hello! I'm working with Career Services who has contracted with a third-party called Burning Glass. Part of the package of what we receive from them is the ability to put widgets into our web pages to help students connect the skills they are learning in their academics and co-curriculars with potential career paths.

For this, they have said that we will need to be able to put a 'code snippet' onto the pages of our website. Here's a brief overview that was sent to us from Burning Glass:

Program Page Codes

You will add a code snippet to each page where the program page widget needs to appear. Each program in the application has its own Program Code, which dictates the program outcomes that appear in the widget. The program Code number in the snippet must match the program page where the iframe is embedded. The list of codes for all of your programs is in the table, where you can choose to copy the code on its own and add it to the snippet, or copy the entire snippet with the correct program code included.

I think this may be what they are talking about: https://www.burning-glass.com/products/career-insight - there isn't a lot of information or any documentation of this publicly available.

@kevincrafts kevincrafts added the new New feature label Jul 22, 2020
@jwfuller
Copy link
Contributor

User wants to embed 3rd party code, specific to client

@jwfuller jwfuller added the 2nd Party JS Request to add extra JavaScript to sites label Aug 11, 2020
@jwfuller
Copy link
Contributor

From @kevincrafts

<div id="ci"></div>
<script src="https://careerinsight.burning-glass.com/enroll/scripts/cienroll.js"></script>
<script>
document.body.onload = function() {
var props = {
accessCode: 'f399bb8',
programCode: 'b1bb493e',
elementId: 'ci'
}
openCI(props);
}
</script>

@kevincrafts
Copy link
Member Author

<div id="ci"></div>
<script src="https://careerinsight.burning-glass.com/enroll/scripts/cienroll.js"></script>
<script>
document.body.onload = function() {
var props = {
accessCode: 'f399bb8',
programCode: 'cb676fc8',
elementId: 'ci'
}
openCI(props);
}
</script>

@kevincrafts
Copy link
Member Author

So I confirmed with the department that each code snippet is program specific. There will need to be a different snippet for each academic program on campus - over 100.

However, they are going to pilot with only a few departments this year, so depending on which departments those are will define how many code snippets will be required. Once the pilot is done they want every academic department to use it.

@kevincrafts
Copy link
Member Author

kevincrafts commented Aug 19, 2020

I'm also concerned with accessibility - I can already see some color contrast issues (they do have a high contrast toggle) but I think we should have the accessibility lab look at this as well.

Screen Shot 2020-08-19 at 8.48.29 AM.png

@kevincrafts
Copy link
Member Author

Confirmed this has gone through an accessibility review and passed. #101106 for reference

@jwfuller
Copy link
Contributor

jwfuller commented Sep 1, 2020

@kevincrafts and I discussed implementing this via Full HTML blocks for the 6 site pilot. If the pilot is a success, we want to implement this as part of a 2nd party content bundle that can be expanded to handle other types of JS driven content

@jwfuller jwfuller added the good first issue Accessible to people that are not deeply familiar with the codebase label Oct 6, 2020
@jwfuller jwfuller added this to To do in Drupal 7 Express Oct 20, 2020
@jwfuller jwfuller added this to the JS includes milestone Oct 22, 2020
@AlanBCole AlanBCole moved this from To do to In progress in Drupal 7 Express Nov 10, 2020
@AlanBCole AlanBCole self-assigned this Nov 10, 2020
@AlanBCole
Copy link
Contributor

@kevincrafts In the javascript includes pr #727 there are two ways to 'include' javascript includes (sorry). Which one is best for this: via context or block placement?

@jwfuller
Copy link
Contributor

For this widget, I think context since it should be applied at the bottom of the page like Google Analytics

AlanBCole pushed a commit that referenced this issue Nov 16, 2020
Added burning_glass to cu_js_includes_enabled in .install
Added burning_glass to  array in .module file at cu_js_includes_block_options()
Added cu_js_includes_burning_glass_form() in .forms.inc file
Added view burning_glass js includes to cu_js_includes_secure_permissions() in .permissions.inc file
Added  to cu_js_includes_get_include_types() in .types.inc file
Added Add Burning GLass Include to [types] array in cu_js_includes_express_node_list_groups() in .types.inc file
Added burning glass php template in /templates/burning_glass.php
@AlanBCole AlanBCole moved this from In progress to Review in progress in Drupal 7 Express Nov 17, 2020
@AlanBCole AlanBCole linked a pull request Nov 17, 2020 that will close this issue
@jwfuller jwfuller moved this from Review in progress to New Issues in Drupal 7 Express Mar 30, 2021
@AlanBCole AlanBCole removed their assignment May 28, 2021
@jcsparks jcsparks moved this from New Issues to Backlog in Drupal 7 Express Jun 8, 2021
@jcsparks jcsparks moved this from Backlog to New Issues in Drupal 7 Express Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2nd Party JS Request to add extra JavaScript to sites good first issue Accessible to people that are not deeply familiar with the codebase new New feature
Projects
Development

Successfully merging a pull request may close this issue.

3 participants