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

Update deprecated notifications API usage #93

Merged
merged 1 commit into from
Feb 26, 2022

Conversation

intgr
Copy link
Collaborator

@intgr intgr commented Feb 20, 2022

Contributor checklist

  • I am targeting the master branch (and not the release branch)
  • I am using the provided codeStyleConfig.xml
  • I have tested my contribution on these versions of PyCharm/IDEA:
  • PyCharm Community 2021.3
  • My contribution is fully baked and ready to be merged as is
  • I ensure that all the open issues my contribution fixes are mentioned in the commit message of my first commit
    using the Fixes #1234 syntax

Description

  • Notification groups are now defined in plugin.xml
  • createNotification() method with over 4 arguments are deprecated
  • Deleted unused methods in Notifications & fixed IntelliJ inspections

Docs: https://plugins.jetbrains.com/docs/intellij/notifications.html#top-level-notifications-balloons

Type of Changes

Type
🔨 Refactoring

* Notification groups are now defined in plugin.xml
* `createNotification()` method with over 4 arguments are deprecated
* Deleted unused methods in `Notifications` & fixed IntelliJ inspections

Docs: https://plugins.jetbrains.com/docs/intellij/notifications.html#top-level-notifications-balloons
@intgr intgr changed the title Fix deprecated notifications API usage Update deprecated notifications API usage Feb 20, 2022
Copy link
Owner

@leinardi leinardi left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@leinardi leinardi merged commit d5824ce into leinardi:master Feb 26, 2022
@leinardi
Copy link
Owner

Hi @intgr, looks like this changes break compatibility with older version of IDEA:
image

To keep them we should raise the min version from 201.8743 to 212.5712.

What do you think?

@intgr
Copy link
Collaborator Author

intgr commented Feb 28, 2022

Ah this isn't something I considered, maybe I should have.

Personally, I would just bump the minimum version and not worry about compatibility. If users don't want to upgrade to newer IDE versions, maybe they shouldn't expect to be able to update plugins either -- they can still use the old plugin version. Similar to how we did in #77 (comment)

Although 212.5712 corresponds to IDEA 2021.2.4, which is still quite new.

Perhaps add a message to the plugin's description or release notes about this: if there are any users who need this plugin in older IDE versions, they can open an issue and we can consider reversing this policy.

@intgr
Copy link
Collaborator Author

intgr commented Feb 28, 2022

By the way, you can change the minimum version in the JetBrains Marketplace website directly.

There is no need to make a new release to change it.

@leinardi
Copy link
Owner

Interesting! I'll try :)

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