Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Fix:#1921 Add CHANGELOG release notes for 1.13 #1924

Merged
merged 3 commits into from
Nov 13, 2019

Conversation

srirambv
Copy link
Contributor

@srirambv srirambv commented Nov 8, 2019

Summary of Changes

This pull request fixes issue #1921

Submitter Checklist:

  • Unit Tests are updated to cover new or changed functionality
  • User-facing strings use NSLocalizableString()

Test Plan:

N/A

Screenshots:

Reviewer Checklist:

  • Issues include necessary QA labels:
    • QA/(Yes|No)
    • release-notes/(include|exclude)
    • bug / enhancement
  • Necessary security reviews have taken place.
  • Adequate unit test coverage exists to prevent regressions.
  • Adequate test plan exists for QA to validate (if applicable).
  • Issue is assigned to a milestone (should happen at merge time).

@srirambv srirambv self-assigned this Nov 8, 2019
@jhreis jhreis added the blocked If a ticket is blocked for some reason, if not using a sub-block label, please provide info in issue label Nov 8, 2019
@jhreis
Copy link
Contributor

jhreis commented Nov 8, 2019

Waiting on @kjozwiak

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
- Added new preview menu on iOS13. ([#1499](https://github.com/brave/brave-ios/issues/1499))
- Added bookmarklet support. ([#1119](https://github.com/brave/brave-ios/issues/1119))
- Updated text size and colours on new tab stats to match desktop. ([#1641](https://github.com/brave/brave-ios/issues/1641))
- Updated StartPage icon. ([#1598](https://github.com/brave/brave-ios/issues/1598))
Copy link
Member

Choose a reason for hiding this comment

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

nit: not everyone knows that StartPage is a search engine. We should add more context. Example:

  • Updated StartPage icon under search engine list.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Startpage shows up as a quick search so dont think if it needs to be explicitly called out.

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@kjozwiak
Copy link
Member

@srirambv mind taking a look when you have some time so we can push these to https://brave.com/latest/?

@srirambv
Copy link
Contributor Author

Fixed. Please review and merge. Didnt change StartPage as its been there in the product already for a long time and users would know about it.

@kjozwiak kjozwiak self-requested a review November 13, 2019 17:25
CHANGELOG.md Outdated Show resolved Hide resolved
@kjozwiak kjozwiak self-requested a review November 13, 2019 17:34
Copy link
Member

@kjozwiak kjozwiak left a comment

Choose a reason for hiding this comment

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

LGTM @srirambv 👍 Awesome job!

@srirambv srirambv merged commit 7402c4d into brave:development Nov 13, 2019
@srirambv srirambv deleted the 131-rel-notes branch November 13, 2019 17:37
@iccub iccub removed the blocked If a ticket is blocked for some reason, if not using a sub-block label, please provide info in issue label Nov 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants