-
Notifications
You must be signed in to change notification settings - Fork 88
Adding "publish-to-gts" script #245
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
Conversation
the `l` got lost in the `.qml` extension...
Build was failing for inconsistent identifier
Removed Linux and MacOs as not tested
regression bug due to changing identifier
Enhancement: style selector Enhancement: separator selector modified: ascii-tree-factory/ASCII-Tree-Factory.qml modified: ascii-tree-factory/info.json modified: ascii-tree-factory/readme.md
renamed: ascii-tree-factory/ASCII-Tree-Factory.qml -> ascii-tree-factory/ascii-tree-factory.qml modified: ascii-tree-factory/readme.md
publish-to-gts/info.json
Outdated
| @@ -0,0 +1,10 @@ | |||
| { | |||
| "name": "Publish to GtS", | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you maybe please use "GoToSocial" instead of "GtS" as name?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, will do
publish-to-gts/publish-to-gts.qml
Outdated
| "identifier": "authCode", | ||
| "name": "Authentication Code", | ||
| "description": "Code returned by GtS after performing a successful authentication, if you paste it here you won't need to authenticate again until expiry", | ||
| "type": "string", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you want to consider string-secret from pbek/QOwnNotes#3150?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to update minAppVersion too if you use the string-secret.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the next iteration will be with string -secret.
|
It would make sense if you rebase your local git repo to |
publish-to-gts/info.json
Outdated
| "platforms": ["linux"], | ||
| "version": "0.1.0", | ||
| "minAppVersion": "24.10.5", | ||
| "description" : "Publish To GtS lets you publish your notes to your GoToSocial account." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe link the name GoToSocial to https://gotosocial.org/?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure
I realized it as soon as I created the pull request. I'm sorry for all these inconveniences, but I'm slowly learning GIT mechanics... not ye used to them! |
|
Should I close this Pull request, adjust things and open a new one? Wouldn't it be cleaner? |
…ization Code script setting.
No, not needed. I'll just squash the commits. |
New script to publish markdown notes to GoToSocial activitypub server.