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

adds giving week banner #415

Merged
merged 3 commits into from
Oct 19, 2023
Merged

Conversation

SBBCornell
Copy link
Contributor

This PR updates user_banner.html with the giving week banner content, and updates the display date range from Monday 10/23 through the end of the day of 10/29 (I set the end date to 10/30, is that right? or should it be 10/29?).

@SBBCornell SBBCornell requested a review from bmaltzan October 19, 2023 17:29
@bmaltzan
Copy link
Contributor

bmaltzan commented Oct 19, 2023

Hey Shamsi,

Yes, I think 10/30 is the right one. The time should default to midnight. You could test by setting BANNER_END to today and tomorrow. And BANNER_START to yesterday.

The BROWSE_USER_BANNER_ENABLED flag was recently created so we could enable the "cloud" messages only in https://browse.arxiv.org/ and not https://arxiv.org/

I think you'll want to remove the two conditions for the aside and slider.css:
config['BROWSE_USER_BANNER_ENABLED']

The banner didn't show up for me locally, until I removed the style="display:none.
I'm loading http://127.0.0.1:8080/static/browse/0.3.4/css/slider.css?v=1.1
but my local config may not be right.

Copy link
Contributor

@bmaltzan bmaltzan left a comment

Choose a reason for hiding this comment

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

Cool, looks like you found some css.
I set an end date, so we don't have to deploy next week.

@bmaltzan bmaltzan added this pull request to the merge queue Oct 19, 2023
Merged via the queue into develop with commit 712366e Oct 19, 2023
4 checks passed
@jonathanhyoung jonathanhyoung deleted the ARXIVCE-846-giving-week-banner-2023 branch November 8, 2023 21:44
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