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

feat(web): add reearth ver support in profile header #428

Merged
merged 4 commits into from May 18, 2023

Conversation

pyshx
Copy link
Contributor

@pyshx pyshx commented May 17, 2023

Overview

This PR introduces functionality to display the current application version in the profile header of the Reearth application, enhancing user awareness of the version they are currently utilizing.

@pyshx pyshx requested a review from KaWaite as a code owner May 17, 2023 15:50
@netlify
Copy link

netlify bot commented May 17, 2023

Deploy Preview for reearth-web failed.

Name Link
🔨 Latest commit ff919e2
🔍 Latest deploy log https://app.netlify.com/sites/reearth-web/deploys/6465b7f2f9ad58000814ea0f

@github-actions github-actions bot added the web label May 17, 2023
@@ -0,0 +1 @@
declare let __APP_VERSION__: string;
Copy link
Member

Choose a reason for hiding this comment

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

We have a

declare global {
    interface Window {
  ..
...
...
}

in our src/config/index.ts file. Not sure it makes sense to put this there too. If it doesn't, perhaps we can move the global declaration in to this new file so that we keep things tidy.
What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right, I may have overlooked that global class exist here. ff919e2 should fix that.

@pyshx pyshx merged commit d1125e7 into main May 18, 2023
2 of 7 checks passed
@pyshx pyshx deleted the feat/support-reearth-ver-in-profile-header branch May 18, 2023 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants