-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: show page name in document title using React Helmet #3229
feat: show page name in document title using React Helmet #3229
Conversation
thanks, @mustajab-ikram for the PR just small chances would be good to have |
fix: Add translation support for page titles
|
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.
thanks @mustajab-ikram for raising the pr
small, enhancement which I did updated the return type of the function and we already have AppLayout so move the logic there
And thanks for sharing the pr :-D working seamlessly
I have completed the task related to Issue #3201. I have added the Page Title functionality using React Helmet as discussed in our previous chats.
![image](https://private-user-images.githubusercontent.com/94978425/256979167-ae217fde-4282-45a2-a400-9da45d6ba73b.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA2ODE3OTcsIm5iZiI6MTcyMDY4MTQ5NywicGF0aCI6Ii85NDk3ODQyNS8yNTY5NzkxNjctYWUyMTdmZGUtNDI4Mi00NWEyLWE0MDAtOWRhNDVkNmJhNzNiLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTElMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzExVDA3MDQ1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWI0ODQzYmJkOTAzN2ViZjI0YzI4OGRkZDVlYWNkNjY1ZmMwYzhiZGU0NGIzNDYxMzk0YmU0MGY1OGM0ZTY5NzImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.DIruCcdi5Bs1d9Xwj_2CH11NL8xozKx4dZ8JC3kFLkw)
Please take a look at the PR when you get a chance. Let me know if there are any further changes or improvements you would like to see.
close #3201