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

Notify user if Navie isn't fully functional #595

Merged
merged 2 commits into from
Mar 1, 2024

Conversation

jansorg
Copy link
Collaborator

@jansorg jansorg commented Feb 28, 2024

Closes #571

If the current IDE's webview JCEF isn't suitable for Navie, then show a notification to ask the user to update. If the user chose "don't show again", then the message is never shown again in any of the user's projects.

We're showing the message on top of the Navie editor tab, but because JCEF rendering is async the message is shown before the webview is fully visible.
If this is a problem, we could inject the message more deeply into the editor, e.g. when the webview's message that it's ready is received.

image

@jansorg jansorg marked this pull request as ready for review February 28, 2024 11:32
Copy link

github-actions bot commented Feb 28, 2024

AppMap runtime code review

Summary Status
Failed tests ✅ All tests passed
API changes 0️⃣ No API changes
Security flaws ✅ None detected
Performance problems ✅ None detected
Code anti-patterns ✅ None detected
New AppMaps ⭐ 2 new junit tests
Removed AppMaps ✖️ 1 removed junit test

⭐ New AppMaps

[junit] Language resolver java project from plugin-core/src/test/java/appland/installGuide/analyzer/LanguageResolverTest.java:22

[junit] Runtime analysis model authenticated without findings from plugin-core/src/test/java/appland/toolwindow/runtimeAnalysis/RuntimeAnalysisModelTest.java:39

✖️ Removed AppMaps

[junit] Default command line service directory tree watched subdir

@jansorg jansorg requested a review from dustinbyrne February 28, 2024 11:45
@ahtrotta ahtrotta merged commit 9b5f184 into develop Mar 1, 2024
6 checks passed
@ahtrotta ahtrotta deleted the jansorg/navie-notification branch March 1, 2024 20:42
appland-release pushed a commit that referenced this pull request Mar 11, 2024
## [0.59.0](v0.58.0...v0.59.0) (2024-03-11)

### Features

-  notify user if Navie isn't fully functional ([#595](#595)) ([9b5f184]())-  support to display the AppMap tool window from the Navie webview ([bb25a8b]())-  Update install guide to include new node instructions ([d7815d7]())

### Bug Fixes

-  compatibility with 2024.1 eap's backwards incompatible API of OAuthRequestHandlerBase ([5fc79c2]())-  open external links with a target attribute like plain external links ([4494a4b]())-  update actions to avoid exception "ActionUpdateThread.OLD_EDT is deprecated and going to be removed soon." with 2024.1 eap ([d6d18ca]())
@appland-release
Copy link
Contributor

🎉 This PR is included in version 0.59.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

apotterri pushed a commit that referenced this pull request Mar 25, 2024
## [0.59.0](v0.58.0...v0.59.0) (2024-03-11)

### Features

-  notify user if Navie isn't fully functional ([#595](#595)) ([9b5f184]())-  support to display the AppMap tool window from the Navie webview ([bb25a8b]())-  Update install guide to include new node instructions ([d7815d7]())

### Bug Fixes

-  compatibility with 2024.1 eap's backwards incompatible API of OAuthRequestHandlerBase ([5fc79c2]())-  open external links with a target attribute like plain external links ([4494a4b]())-  update actions to avoid exception "ActionUpdateThread.OLD_EDT is deprecated and going to be removed soon." with 2024.1 eap ([d6d18ca]())
apotterri pushed a commit that referenced this pull request Mar 28, 2024
## [0.59.0](v0.58.0...v0.59.0) (2024-03-11)

### Features

-  notify user if Navie isn't fully functional ([#595](#595)) ([9b5f184]())-  support to display the AppMap tool window from the Navie webview ([bb25a8b]())-  Update install guide to include new node instructions ([d7815d7]())

### Bug Fixes

-  compatibility with 2024.1 eap's backwards incompatible API of OAuthRequestHandlerBase ([5fc79c2]())-  open external links with a target attribute like plain external links ([4494a4b]())-  update actions to avoid exception "ActionUpdateThread.OLD_EDT is deprecated and going to be removed soon." with 2024.1 eap ([d6d18ca]())
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.

Notify the user to upgrade if they're on <= 2022 to use Navie
3 participants