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.

Sign up for GitHub

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

Jump to bottom

Rename Brave Today to Brave News in code #28575

Closed
fallaciousreasoning opened this issue Feb 17, 2023 · 0 comments · Fixed by brave/brave-core#17265
Closed

Rename Brave Today to Brave News in code #28575

fallaciousreasoning opened this issue Feb 17, 2023 · 0 comments · Fixed by brave/brave-core#17265
Assignees
@fallaciousreasoning
Labels
OS/Android Fixes related to Android browser functionality OS/Desktop QA/No release-notes/exclude
Milestone
1.50.x - Release

Comments

@fallaciousreasoning
Copy link

This is follow up to #15925

We should rename all the internal variables/classes/folders from braveToday to braveNews so it matches the UI and is consistent across brave-core.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fallaciousreasoning fallaciousreasoning

Labels
OS/Android Fixes related to Android browser functionality OS/Desktop QA/No release-notes/exclude
Projects
None yet
Milestone
1.50.x - Release
Development

Successfully merging a pull request may close this issue.

Rename Brave Today to Brave News brave/brave-core
2 participants
@fallaciousreasoning @brave-builds