mirror of
https://github.com/ProxymanApp/Proxyman.git
synced 2026-04-26 08:35:53 +03:00
[GH-ISSUE #797] Script export includes all imported datafiles, not just the ones required by the scripts being exported. #792
Labels
No labels
Discussion
Feature request
In Progress...
Plugins
Waiting response
Windows
Windows
bug
duplicate
enhancement
feature
good first issue
iOS
macOS 10.11
question
wontfix
✅ Done
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/Proxyman#792
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @mprice-banno on GitHub (Feb 20, 2021).
Original GitHub issue: https://github.com/ProxymanApp/Proxyman/issues/797
Originally assigned to: @NghiaTranUIT on GitHub.
Proxyman version? (Ex. Proxyman 1.4.3)
2.17
macOS Version? (Ex. mac 10.14)
10.15.6
Notes
If you have imported files into various scripts, and then are exporting a script to share, the exported config file includes all the data files you have ever imported. Although this does match the "Export Scripts and All Imported Files" This leads to two issues, potentially very large exported config files, and sharing files that you didn't intend or want to.
I would expect that only the data files that are references via a
require()would be exported. So for example, if I am exporting two scripts, and each script references a unique data file, only those two data files would be included in the export.Another option is to have the user decide which data files they want to include in the export, although doing it automatically based on the
require()would be ideal.This is probably more of a feature request than a bug, although I think people wouldn't expect all the data files to be exported.
Steps to reproduce
Unzip and look at the exported config file, and you'll see that all the imported files have been included,
Expected behavior
Only the ones referenced by the exported scripts should be in the config file.
Screenshots (optional)
@NghiaTranUIT commented on GitHub (Feb 20, 2021):
Thanks for opening the ticket @mprice-banno 👍
It's the first limitation of the Export/Import Script Rules that we're facing.
I would like to share the road map of the Export/Import Scripting Rules:
require()of each scripting file. It's what we expect 😄I will allocate resource to implement the v2 soon 🌮
@mprice-banno commented on GitHub (Feb 22, 2021):
👍 Thanks @NghiaTranUIT you are doing a fantastic job with this tool!