-
Notifications
You must be signed in to change notification settings - Fork 100
[USH-1679] Rename the heading of the new post screen to 'Add New Post' #1482
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
@@ -12,7 +12,7 @@ const routes: Routes = [ | |||
{ | |||
path: 'create/:type', | |||
component: PostEditComponent, | |||
data: { breadcrumb: 'Create Post' }, | |||
data: { breadcrumb: 'Add New Post' }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can just have another ticket to add translation for the breadcrumb text in this file, similar to how it's done in app routing module file @MMwandigha
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Ifycode Oh yeah its true... I also thought of the same. I noticed that all the breadcrumb texts in the post-routing module haven't been translated yet. I shall create one
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏼
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here's the associated PR for this:
#1483
Renaming the heading of the new post breadcrumb to 'Add New Post'
Testing
Previously it was 'Create Post' but now it's 'Add New Post'