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

Implement Tenor client api #13816

Conversation

ngoctr
Copy link
Contributor

@ngoctr ngoctr commented Apr 6, 2020

This PR implement Tenor client API as part of 13803

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered adding accessibility improvements for my changes.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Giorgio Ruscigno and others added 30 commits March 28, 2020 08:44
…to load ReaderTabView

- Refactored ReaderTabView to use let constants for objects that should be immutable
… the divider to avoid temporary constraint breaks
- Update ReaderTabViewModel.swift, add a Tab Bar extension to sync topics with the backend in Core Data, and fetch them to populate the tab bar items
- Update ReaderTabView, use the reader tab bar items retrieved from the view model
- Update feature flag
Also update Aztec to version 1.17.1
- Update FilterTabBar.swift, re-activate tabBarHeightConstraint after changing height value
- Update ReaderTabView.swift, re-arranged properties by type
- Update ReaderTabViewController.swift, removed nibName and bundle from init signature, set them to nil by default in super.init() call
- Update ReaderHelpers.swift, add some clarifying comments in rearrange method
This avoids a potential crash when the view has not yet fully loaded
mindgraffiti and others added 28 commits April 3, 2020 09:42
…berg/integrate_release_1.25.0

Update Gutenberg to 1.25.0 release
Colors are now inline with design and change according to light/dark mode.
…re/Reader-tabs-navigation

Uses reader Topics to populate the new Reader Tab Bar
…review-presentation-size

Change iPad Post Preview presentation to fullscreen
…3735-searchcontroller-crash

Safely unwrap searchController in Posts List
…/remove-segue-show2FA

Remove the .show2FA segue and replace with programmatic nav
…edia-selection-via-other-apps

Display Cancel button in "Other Apps" media browser
@peril-wordpress-mobile
Copy link

Warnings
⚠️ PR is missing at least one label.
⚠️ PR is not assigned to a milestone.
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS

@ngoctr ngoctr closed this Apr 6, 2020
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.

6 participants