Skip to content

feat: about page conversion#393

Merged
grudra7714 merged 4 commits intofeat/vue3-upgradefrom
feat/about-page
Apr 29, 2025
Merged

feat: about page conversion#393
grudra7714 merged 4 commits intofeat/vue3-upgradefrom
feat/about-page

Conversation

@grudra7714
Copy link
Copy Markdown
Contributor

Summary

Converts about.vue page to vue3 composition api

Fixes https://krumtest.atlassian.net/browse/EPINIO-96?atlOrigin=eyJpIjoiOWI1MGIyMmM3ZmQ2NDM0YmFiZDU3YjQ2NjhhYjJlYjIiLCJwIjoiaiJ9

Occurred changes and/or fixed issues

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Signed-off-by: Rudra Gupta <grudra7714@gmail.com>
Signed-off-by: Rudra Gupta <grudra7714@gmail.com>
@grudra7714 grudra7714 requested a review from dcharles525 April 28, 2025 18:37
Copy link
Copy Markdown
Member

@dcharles525 dcharles525 left a comment

Choose a reason for hiding this comment

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

I am getting "store is undefined" errors on line 16/17. I think this is also causing the computed values (appName, versionString) to not resolve and display in the UI.

image

Signed-off-by: Rudra Gupta <grudra7714@gmail.com>
@grudra7714
Copy link
Copy Markdown
Contributor Author

image

@grudra7714 grudra7714 requested a review from dcharles525 April 29, 2025 14:47
Signed-off-by: Rudra Gupta <grudra7714@gmail.com>
@grudra7714
Copy link
Copy Markdown
Contributor Author

image

Copy link
Copy Markdown
Member

@dcharles525 dcharles525 left a comment

Choose a reason for hiding this comment

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

LGTM!

@grudra7714 grudra7714 merged commit 3d471d1 into feat/vue3-upgrade Apr 29, 2025
0 of 2 checks passed
@krumware krumware added the kind/enhancement New feature or request label May 29, 2025
@dcharles525 dcharles525 deleted the feat/about-page branch October 16, 2025 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants