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

Library: Port 'Toasts' to Python #732

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Library: Port 'Toasts' to Python #732

wants to merge 2 commits into from

Conversation

UrtsiSantsi
Copy link
Contributor

No description provided.



action_console.connect("activate", on_activate)
workbench.window.add_action(action_console)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm getting this error here:

image

Is this because of #110?
The rest of the demo seems to work.

Copy link
Contributor

@theCapypara theCapypara left a comment

Choose a reason for hiding this comment

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

Hi! Thank you! The demo unfortunately does not launch for me:

Traceback (most recent call last):
  File "/app/share/re.sonny.Workbench.Devel/gdbus_ext.py", line 319, in on_method_call
    result = method_func(obj, *args)
             ^^^^^^^^^^^^^^^^^^^^^^^
  File "/app/share/re.sonny.Workbench.Devel/python-previewer.py", line 135, in run
    spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/marco/.var/app/re.sonny.Workbench.Devel/data/re.sonny.Workbench.Devel/sessions/2023-11-11 17-13-05/main.py", line 52, in <module>
    workbench.window.add_action(action_console)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'Window' object has no attribute 'add_action'

That part is probably not supportable yet in Python either, see also the Vala code and the issue it's refering: #110

@UrtsiSantsi
Copy link
Contributor Author

That's exactly what I said above 😄.
So what do we do:

  1. Since the demo is still useful without this action comment out the line and mention Allow using actions in the external preview #110
  2. Turn this into draft until Allow using actions in the external preview #110 is resolved

@sonnyp sonnyp marked this pull request as draft November 12, 2023 10:58
@sonnyp
Copy link
Contributor

sonnyp commented Nov 12, 2023

I have converted to draft.

@theCapypara
Copy link
Contributor

Oh, hm! Sorry about that, I think GitHub didn't show me your comments yesterday 🧐

@sonnyp
Copy link
Contributor

sonnyp commented Mar 23, 2024

@UrtsiSantsi could you reopen this against https://github.com/workbenchdev/demos ?

I think we should move forward and just disable the action button for now. Like Vala

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants