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

New option to upload/POST URLs to a web service/end point #27

Open
dubiouscript opened this issue Dec 16, 2023 · 4 comments
Open

New option to upload/POST URLs to a web service/end point #27

dubiouscript opened this issue Dec 16, 2023 · 4 comments
Labels
help wanted Extra attention is needed question Further information is requested

Comments

@dubiouscript
Copy link

dubiouscript commented Dec 16, 2023

Original title: upload option

(i thiink!?) iv attached a patch
created from alct/export-tabs-urls
with (verry basic) modifications to add 'upload' functionality
export_tabs_urls_and_titles-diff.txt

... tested with

shell2http -log=s2h-log.txt -cgi / 'echo "Access-Control-Allow-Origin: *\n" ;echo foo ; cat - | tee ./svr-log000.txt >/dev/tty ; '
msoap/shell2http#99 (comment)

could this or something similar be considered for addition to export-tabs-urls-phoenix ?

@clach04
Copy link
Owner

clach04 commented Dec 16, 2023

Thanks for taking a look at this and making time to work on a code implementation @dubiouscript !

I'll take a look at the patch when I get access to a desktop/laptop. I don't understand the use case just yet, I'm hoping the patch will help me understand more.

@clach04 clach04 changed the title upload option New option to upload/POST URLs to a web service/end point Dec 16, 2023
@clach04
Copy link
Owner

clach04 commented Dec 16, 2023

@dubiouscript Right now this isn't suitable to merging in. Potentially with some polish I could see this being added but it isn't something I'm interested in spending time working on. It isn't something I would be testing.

Questions for you:

  1. Did I understand the aim of this change (does the new title of this issue reflect what you want)?
  2. Does this require new permission(s) to be granted in the web browser?
  3. Are you interested in polishing this?
    a. the URL needs to be a config option
    a. I do not think many end users will use this so I think the button should be hidden by default, for example exposed if config is set.

No worries if you don't have time, interest in pursuing. If this works for you can always load it in debug mode.

@clach04 clach04 added help wanted Extra attention is needed question Further information is requested labels Dec 16, 2023
@dubiouscript
Copy link
Author

  1. title is better
  2. not afaik ,... but *
  • im runinng the old plugin on old browser
    & this(new/-phoenix) plugin dont support old browser
  1. yes when i have upgraded ect i will (hopefuly) come back to this , and send a patch which addresses a & b .

No worries if you don't have time, interest in pursuing. If this works for you can always load it in debug mode.

just wanted to make patch/work some what avalable (in (the unlikely)case anyone was searching for similar)

@clach04
Copy link
Owner

clach04 commented Feb 2, 2024

@dubiouscript take a look at the branch https://github.com/clach04/export-tabs-urls-phoenix/tree/manifest_v2_legacy that maybe what you want. It tracks the main branch but with a v2 manifest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants