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

Fixes Tool Page Title #119

Merged
merged 2 commits into from
Mar 28, 2022
Merged

Fixes Tool Page Title #119

merged 2 commits into from
Mar 28, 2022

Conversation

aryehraber
Copy link
Contributor

It seems PR #108 tried adding a Page Title to the Backup Tool, however it doesn't work due to a missing reference to the __() method.

This is the error I'm seeing in Nova when visiting the Backup Tool:
Screenshot 2022-01-13 at 09 49 45

Since the __() is bound to the Vue instance, it needs to be this.__(). Let me know if there are any questions, thanks!

@aryehraber
Copy link
Contributor Author

Hi Spatie team, please let me know if there's any feedback to get this merged, happy to make changes is needed!

@freekmurze
Copy link
Member

Could you resolve the conflicts?

@aryehraber
Copy link
Contributor Author

Sure thing, must be due to the other PR that was just merged -- will get sorted asap!

@aryehraber
Copy link
Contributor Author

Alright, rebased from master and rebuilt the assets -- should be good to go now!

(not sure why the tests are failing, don't think this has to do with the PR)

@freekmurze freekmurze merged commit d305afb into spatie:master Mar 28, 2022
@freekmurze
Copy link
Member

Thank you!

@aryehraber aryehraber deleted the fix/page-title branch March 28, 2022 12:16
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.

2 participants