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

Demo Box - Update Vocabs and Styling #3945

Closed
4 tasks done
joncameron opened this issue Jan 17, 2020 · 3 comments
Closed
4 tasks done

Demo Box - Update Vocabs and Styling #3945

joncameron opened this issue Jan 17, 2020 · 3 comments
Assignees

Comments

@joncameron
Copy link
Contributor

joncameron commented Jan 17, 2020

Description

Making our demo site and the content within look nice and make sense for navigation, so that users evaluating Avalon can quickly see what Avalon is like and how it might be set up.

Done Looks Like

  • Add featured collections
    • home_page:
      • featured_collections: ['8910jt56k', '3197xm04j']
      • carousel_collections: ['8910jt56k', '3197xm04j', '0v8380680', 'dj52w479s']
  • Set these to be the Units in controlled_vocabulary.yml:
    • Test Content From Evaluators
    • Avalon Media System
    • Samvera Community
    • Sample Content
  • Remove "The" from "The Avalon Media System" (Remove Article #3950)
  • Update https://www.avalonmediasystem.org/try-out-avalon page after Demo is updated
@phuongdh
Copy link
Member

phuongdh commented Jan 24, 2020

Given that we want to pull in a few commits from develop, I wonder if it makes sense to

  • cut a new demo branch from master and cherry-pick the commits
  • or make a new release?

@phuongdh
Copy link
Member

Collections Carousel not working:

Error: URL contains XSS injection attempt
    at a (https://demo.avalonmediasystem.org/packs/js/application-174e572ab1eaef31ba71.js:1:554329)
    at https://demo.avalonmediasystem.org/packs/js/application-174e572ab1eaef31ba71.js:1:554769
    at https://demo.avalonmediasystem.org/packs/js/application-174e572ab1eaef31ba71.js:1:358122
    at new Promise (<anonymous>)
    at e.exports (https://demo.avalonmediasystem.org/packs/js/application-174e572ab1eaef31ba71.js:1:357089)
    at e.exports (https://demo.avalonmediasystem.org/packs/js/application-174e572ab1eaef31ba71.js:1:552172)

@phuongdh
Copy link
Member

Turns out it's an Axios bug that rejects query strings starting with "on" like our carousel endpoint here https://demo.avalonmediasystem.org/collections.json?limit=10&only=carousel

axios/axios#2646

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

No branches or pull requests

2 participants