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

tweak social card meta yet again #2228

Merged
merged 1 commit into from
Dec 18, 2023
Merged

Conversation

bnewbold
Copy link
Contributor

  • handle missing content (PWI opt-out) better
  • use opengraph "article" and "profile" types, not just website for everything
  • fix timestamps, and used indexed at not created at
  • remove site-level twitter handle from most card meta

This is golang HTML template code (using django/jinja2 template language), but I hope should be easy to review what it is doing.

- handle missing content (PWI opt-out) better
- use opengraph "article" and "profile" types, not just website for everything
- fix timestamps, and used indexed at not created at
@bnewbold bnewbold merged commit 8da5a37 into main Dec 18, 2023
4 checks passed
@bnewbold bnewbold deleted the bnewbold/tweak-social-cards branch December 18, 2023 16:24
estrattonbailey added a commit that referenced this pull request Dec 18, 2023
* origin/main: (22 commits)
  basic public RSS feed for profiles (#2229)
  use `s.likeColor` everywhere (#2234)
  Add credits to localization doc (#2233)
  Fix `Logotype` fill for dark mode (#2230)
  Hide label appeal on self-labeled posts (#2232)
  Fix & Add: Japanese Translation (3) (#2226)
  tweak social card meta yet again (#2228)
  bskyweb: update golang indigo dep (from May!), and some small devex tweaks (#2227)
  Fix: Some display issues (#2219)
  Update splash screen tagline, update translations, bump ios build number
  Bump ios build number
  Tweaks (#2225)
  1.60
  Traffic reduction and tuned caching strats (#2215)
  Super secret changes don't look (#2218)
  Fix android icon dims (#2213)
  Log post creation failures (#2205)
  Some brand, some pwi (#2212)
  🤫  (#2211)
  PWI behavior updates (#2207)
  ...
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