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

#88 add custom text box to excel skeleton #89

Closed
wants to merge 1 commit into from

Conversation

jckwik
Copy link
Contributor

@jckwik jckwik commented Mar 20, 2023

As a solution to #88 I've created a custom text box for our excel skeleton. The features include the following:

  • Just load the text box once and the text box takes care of its own management
  • The text box has a number on it to track how many times you have entered input (1-6) and automatically closes after 6 entries, clicking cancel, or clicking OK with nothing in the box
  • Handles multiple inputs on newlines, allowing for mass data entry for testing (tested up to 300 lines)
  • (Recent bugfix) Handles one line of input and a newline character as one line of input
  • (Known issue) Errors on no input and a newline character

I wasn't able to make a branch directly in the base repo, so my suggestion would be do so and edit this PR to point there so that you're not immediately overwriting main, or else I can make another commit with the skeleton as a different name.

@fuddster
Copy link
Collaborator

fuddster commented Jan 9, 2025

Thank you for this pull request. We've moved away from using text boxes and custom VBA parsing functions. We now use tsv = Tab Separated Values so that the data can be input directly into the spreadsheet without need for input boxes or custom VBA macros.

Thank you for using and contributing to ScoutingPASS

@fuddster fuddster closed this Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants