Skip to content

Adding some new usable bookmark content tags:#1

Merged
w4tchdoge merged 2 commits into
w4tchdoge:mainfrom
PrincessGrouchy:p-rebecca-AO3-bookmark-changes
Nov 15, 2025
Merged

Adding some new usable bookmark content tags:#1
w4tchdoge merged 2 commits into
w4tchdoge:mainfrom
PrincessGrouchy:p-rebecca-AO3-bookmark-changes

Conversation

@PrincessGrouchy
Copy link
Copy Markdown
Contributor

  • expanding the available formatting for relationship tags
  • adding character tags
  • adding work bookmark count tags

- expanding the available formatting for relationship tags
- adding character tags
- adding work bookmark count tags
@w4tchdoge
Copy link
Copy Markdown
Owner

Thank you for the pull request! Sorry I didn't see this earlier but I've been busy with university so I haven't really been looking at GH notifications. That also means I can't do a code review on this PR right now but I'll try to when I get some free time (hopefully over the weekend).

- moved where some of the workInfo variables are defined so that it
  makes more sense in my head

- renamed some variable to better describe what they are

- renamed the output workInfo vars for relationships and characters to
  bring them in like with the naming used for the freeform vars

- renamed the intermediate variables used in defining the relationships
  and character vars to bring them in line with how they're named for
  the freeform vars

- added de-duping for the new variants of the relationship tags
  for when the script is run on a series page

- cleaned up the formatting in the area where all the workInfo vars
  are described

- added a debug function which prints all the workInfo variables to
  the console using a dictionary that contains all the workInfo vars
@w4tchdoge w4tchdoge merged commit ecfdf57 into w4tchdoge:main Nov 15, 2025
@PrincessGrouchy
Copy link
Copy Markdown
Contributor Author

Thank you so much for reviewing and merging! I use your script extensively and thought you might enjoy some of my changes.

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