Skip to content

add deploy instructions - #1047

Merged
siwelwerd merged 1 commit into
mainfrom
StevenClontz/20260808/update-2026ed-precalc-bank
Aug 10, 2026
Merged

add deploy instructions#1047
siwelwerd merged 1 commit into
mainfrom
StevenClontz/20260808/update-2026ed-precalc-bank

Conversation

@StevenClontz

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown

🚀 Preview available 🚀

https://d80a572e.tbil.pages.dev

@StevenClontz

Copy link
Copy Markdown
Contributor Author

self-merging as technical documentation update

@StevenClontz
StevenClontz requested a review from siwelwerd August 8, 2026 14:09
@StevenClontz

Copy link
Copy Markdown
Contributor Author

Well I guess I don't have that superpower (easily) here, so tagging @siwelwerd lol

@StevenClontz StevenClontz added documentation Improvements or additions to documentation technical Requires modifying project's technology infrastructure or non-trivial programming labels Aug 8, 2026
@siwelwerd

Copy link
Copy Markdown
Contributor

Who is the audience for this documentation? My guess is its about 2 or 3 people, in which case I wonder if a tab in the README on the front page of the repository is the place for it.

Alternative suggestions are a lowercase filename (so Github doesn't make a tab for it) or in the wiki.

@StevenClontz

Copy link
Copy Markdown
Contributor Author

I have no problem making lowercase so it doesn't become a tab (I didn't realize that was the behavior -- I thought only a finite list of filenames got tabs), but I would prefer not to have to search the git history to find where this info was deleted from the README in the future. (-: And I'd rather be able to do it from my dev environment, without going out to the wiki.

@StevenClontz

Copy link
Copy Markdown
Contributor Author

I don't think DEPLOYING.md gets a tab:

image

@StevenClontz
StevenClontz marked this pull request as draft August 8, 2026 16:59
@StevenClontz

Copy link
Copy Markdown
Contributor Author

I built from this branch, but somehow it didn't build the missing outcome. Investigating.

@StevenClontz

StevenClontz commented Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

Looks like it was probably my user error in the upload to AWS -- I see the build did work locally. Leaving this open and draft until I get it fixed (but my plane is finally boarding).

@siwelwerd

Copy link
Copy Markdown
Contributor

I don't think DEPLOYING.md gets a tab:

Oh, perfect then. I had internalized that the capital letter ones got tabs.

I would prefer not to have to search the git history to find where this info was deleted from the README in the future.

It was deleted as part of #925 after languishing for a few months awaiting review 🫠

@StevenClontz

Copy link
Copy Markdown
Contributor Author

I think the problem earlier was just an issue of caching. I can confirm https://library.tbil.org/2026/precalculus/exercises/#/bank/TR4/1/ is working.

@StevenClontz
StevenClontz marked this pull request as ready for review August 9, 2026 14:10
@StevenClontz

Copy link
Copy Markdown
Contributor Author

It was deleted as part of #925

Yeah, I'd clocked that. I like having it tucked away like this rather than being on the front README.

@siwelwerd
siwelwerd merged commit b939ba5 into main Aug 10, 2026
4 checks passed
@siwelwerd
siwelwerd deleted the StevenClontz/20260808/update-2026ed-precalc-bank branch August 10, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation technical Requires modifying project's technology infrastructure or non-trivial programming

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants