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

[fix] Replace triptych video with static only #1880

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

hannahpurcell
Copy link
Contributor

Replaced the old "closing doors" video with a "don't scroll & stroll" static content Karti shared from CX.

@hannahpurcell hannahpurcell requested review from a team and jzimbel-mbta and removed request for a team September 29, 2023 19:36
@github-actions
Copy link

Coverage of commit 5b38ec0

Summary coverage rate:
  lines......: 40.4% (2296 of 5684 lines)
  functions..: 41.8% (1051 of 2512 functions)
  branches...: no data found

Files changed coverage rate: n/a

Download coverage report

@hannahpurcell hannahpurcell merged commit 62a359b into master Oct 2, 2023
4 checks passed
@hannahpurcell hannahpurcell deleted the hp/replace-cx-video branch October 2, 2023 15:26
jzimbel-mbta added a commit that referenced this pull request Oct 18, 2023
* Move OFM app HTML out of priv/static and into version control; Update packaging instructions with additional steps to minimize package size

* Bump triptych version

* tweak: Adjust routes/endpoints to accept error logs from OFM clients

* Add and use OFM app error boundary component

* Prevent OFM util fns from swallowing errors so the boundary can catch and log them

* Fix incorrect fn name in packaging READMEs

* Bump triptych version

* Completely remove closing-doors PSA set

* Convert all rgba values to hex in CSS used by triptych app

* Remove animation on "You are here" arrow :(

* Use no-data graphic for triptych app preview image, since apparently it's used as a fallback

* tweak: Label lowest crowding level as "Seats available" (#1881)

* Replace triptych video with static only (#1880)

---------

Co-authored-by: Hannah Purcell <[email protected]>
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