These two variables should be user properties to avoid repeating expensive read commands.

There are a number of #REF! errors in the 1822MX spreadsheet template. These should be fixed. Also, there is a line of code referencing these existing #REF! errors that should be removed/changed afterwards.

There is an issue with the CNoR concession not being found if it is listed in a bid box during an SR. This is because CNoR is not capitalized in the game data, but we convert major names to all uppercase to compare them to the game data list.

The current treasury needs to be zeroed out to make the total bank correct.

After a new sheet is created, formula cells should be automatically protected.

The alert for printing errors in showResults() should include a title and possibly an OK button.

If you have fewer than seven players, the nonexistent players have negative certificate counts. This seems to come about because they are the presidents of all unfloated minors and majors (since the empty string matches the empty string).

It makes sense to deduce winning bids from current cash, but the money should be accounted for somewhere.

Trains exported during an SR shouldn't necessarily increment the phase by 1. It could be the case that the user forgot to increment the phase previously, so the phase must be changed to match the type of train that was just exported.

Investigate a better way to calculate a cell's reference for use in the INDIRECT() function. This should be possible by using CELL() to get the cell's address.

There should not be a dropdown in the "new round" dialog to select the game. This should be automatically detected using information from the one of the tabs (probably the Setup tab).

Investigate more efficient ways to detect OR/SR in the round.html dialog. Currently, an expensive read query is made each time a new dropdown item is selected.

If we delete the minor in, say, BZ2, it messes up the coloring in the minor draw pile.

In the 1822MX setup tab, the three minors that can be removed for NdeM are M14, M16, and M17, when they should be M14, M15, and M17.

In our current 1822MX game, NdeM was left with five trains after SR2, despite the train limit being four. Perhaps trains are being rusted before the extra bid box 1 train is exported to NdeM.

After generating a round, there should be an option to see a printout of the player/company order for the new round. This is particularly useful for play-by-post games. It's more or less impossible to get perfectly right for operating rounds, because company share prices can be the same without any way of knowing from the spreadsheet which company should be first. However, we can easily generate a correct or nearly correct ordering.

It would be nice if the new round script would lock the previous tab when making a new one. We've had lots of issues with players inadvertently entering things on old tabs. This code, added at line 1163 in 1822.gs, is working well for us.

curSheet.protect().setWarningOnly(true);

If a minor is acquired in an X.1 OR, our practice is to place a "minor closed" comment (along with a "shares to XX" in the major). If going into an SR this works fine, but if creating a new OR, the sheet hides the minor even though there are still shares.

Currently, any previous sidebars will remain open when a new round dialog is created. Any open sidebars should be closed when this dialog is opened.

When generating a new OR from an OR, the generated outline does not necessarily sort majors by stock price order. As an example, see OR9.1 in our current 1822CA game.

Hi,

Me and a couple of friends are designing our own 18xx game loosely based on 1822 (18OX). To carry on playtesting in the current mess, we're trying to go online and get things set up for online play using Board18. A spreadsheet is a requirement for that. Would it be OK if we forked your 1822 spreadsheet and adapted it to our game? We'd be very grateful and are happy to give credit to you for the original spreadsheet and link to your work for 1822 if we make 18OX public.

Thanks.

The stock round summary should ignore concessions in phases 5-7, rather than treat them like certificates that players can bid on.

The README images need to be updated after the new round dialog UI is changed.

Currently, when creating a new sheet, cell protections are automatically copied from the older sheet. This dominates the script's runtime, and in some cases, may not even be desired behavior. There should be an option for the user to select whether or not they would like to copy protections from the previous sheet.

The detectRound() function should not use hardcoded row and column values to get the round status of a page.

In the 1822CA sheet, the CP president's share is counted as a double share in the certificate count. (The cells AQ81-87 have the same logic to deal with a president's share as the other major columns).

The script should also be able to generate rounds for an 1822CA spreadsheet.

The Results object should have a show() method. Also, checkIndex() should immediately call this method if the index is -1.

There should be a separate CSS file to avoid inline CSS.

If the script encounters an error, an error dialog should be displayed to the user. Currently, the script may just hang without any indication that it encountered an error. The error should suggest that users open a GitHub issue to report the problem.

Currently, stock price increases for sold-out majors are noted in the reminders when generating an OR from an SR. These increases should also be noted in the summary portion of the generated outline.

Many Javascript functions should be removed from 1822mx.gs and placed in a separate file.

Currently, we use two lists to print the outline: summary and reminders. This has led to some undesirable formatting problems when generating the round summary. Instead, there should be a third list added so it is easier to keep track of when the outline should use newlines. The structure should be:

Name of new round
    Summary

    Order outline

    Reminders

The 1822CA template lists the name of each company's home station, but does not list the cell. This should be added, especially because the map is very large.

Currently, the option for generating a new round is embedded into the menu bar, which isn't available in the Google Sheets app for iPhone (unsure about iPad). This may be better as a button. It would simplify the "new round" dialog, as the game name and current round name would be implied by the spreadsheet calling the function.

We can currently attempt to discard invalid train types (like "M"). The result is an error before the script has a chance to gracefully return the "invalid trains" message. We should:

• Not even try to discard trains if trainErrors.length > 0
• Add additional logic so only valid trains (L, 2, 3, ..., 7) can be discarded

Currently, user properties are stored as several different objects. These should be stored as a single object.

Apparently, multiple tabs are opened when creating a new round in Safari.

The Misc player income/expense cells are not cleared out after a new round is generated. This holds for 1822CA; I'm not sure about 1822MX.

Include description, links to files, setup instructions, etc.

The starting phase cell in the spreadsheet should not need to be updated. It can just indirectly refer to the previous sheet's ending phase.

The 1822MX and 1822CA templates have old, obsolete information in their respective 'Extra Data' sheets. These should be replaced with clear instructions for using the sheet, where this repo is located, etc. We should also consider adding a randomizer to auto-populate the Base sheets.

In the templates, the Director row should be filled with automatic formulas to display the name of a company director.

It would be nice if only the active minor columns were visible in the spreadsheet: it would save a lot of scrolling. I think only the minors that have been in the bidding boxes should be visible. If you like, I could work on how to do it in the scripts.

The 1822MX private descriptions (and even assigned numbers) were recently changed in the rules. The spreadsheet should be updated for this.

When creating a new 1822CA OR from SR, spending by the tax haven under the "Stock" column is not cleared out.

Update each script to the new Apps Script runtime. This allows the use of let/const, arrow functions, default parameters, multiline strings, and more. Update the code for these features in obvious places.

Only affects a six-player game, and only because the bank size ends up wrong. But still...