search

ynput / ayon-applications

AYON addon to maintain applications
Apache License 2.0
4 stars 5 forks source link

add missing mayapy setting #10

Closed MustafaJafar closed 3 months ago

MustafaJafar commented 3 months ago

Changelog Description

Add missing mayapy setting.

Additional info

I don't know if we should fix the missing mayapy setting or remove its definition from applications.json

Testing notes:

  1. You should find mayapy in settings
  2. Maybe try launching it from launcher (IDK)
MustafaJafar commented 3 months ago

⚠️ I'm not sure whether it makes sense to have mayapy there as an individual application entry - to me it's more clutter than useful but I'll leave that to someone else to decide. @antirotor @iLLiCiTiT

I don't really see why one would manage that in complete isolation of the maya environments.

Other than that the fix makes sense.

if they mayapy were removed from applications settings on purpose as Kuba mentioned in discord, then should we remove the mayapy from the json file?

BigRoy commented 3 months ago

if they mayapy were removed from applications settings on purpose as Kuba mentioned in discord, then should we remove the mayapy from the json file?

Yes.

MustafaJafar commented 3 months ago

closing this PR in favor of https://github.com/ynput/ayon-applications/pull/11