-
Notifications
You must be signed in to change notification settings - Fork 48
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
feat: allow dragging sas content into editor #510
Conversation
6715b3a
to
53daf34
Compare
It failed to run the generated FILENAME statement if the fileref name having more than 8 bytes. And fileref name must conform to the rules for SAS names. |
@Zhirong2022 Can you provide an example of an invalid file name? Also, do you know where the "rules for SAS names" might be (like, is it documented somewhere)? |
I scrapped the file name generation part of this in favor of using a tab stop to fill in the fileref. Now, when a user drags a file into an editor, we'll automatically select/place a cursor at fileref where they can rename as needed. |
|
This has been addressed in 214d11e. That said, a sas program like the following will still not run:
I attempted to find a workable solution by using the file step in studio to produce code. The code it produced is similar to the code above, and both failed to run with this error:
All this is to say, I don't think there's anything I can do in this PR to address, but I just wanted to bring it up. |
Well, it might run into such kind of scenarios if the file/folder name contains quotation marks. Ideally, it would be perfect to have a statement without any errors. The user can fix it manually. Normally, we do not suggest to new a file/folder having special characters otherwise it will cause some troubles unexpectedly. |
The filepath is not changed accordingly in filename statement in certain case, please follow the steps below to reproduce 1.Create a new file folder named like 'NewFolderTest' under My Folder, new a .sas file named like 'NewFileTest.sas' under 'NewFileFolderTest'. |
This was due to a little cache that I put together that wasn't really saving us that much. I've since removed the cache and things should be working as expected now. |
0747096
to
df0affe
Compare
The feature works as expected. |
Summary
This adds the ability to drag sas content into sas programs. See #329 for more details
Testing