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

another sync of changes from my other repo #171

Merged

Conversation

cooljeanius
Copy link
Collaborator

@cooljeanius cooljeanius commented Jan 27, 2024

  • create CODEOWNERS file, to ensure that the primary people who have edited files get notified when changes to them are proposed (I'm the only one in it so far, as anyone else who it'd make sense to add would have to be added as a collaborator to this repo first)
  • enable optional advancements; closes Enable optional advancements? #118
  • confirmation of @alxrem's translation credits entry
  • start on ChangeLog entries for next release (which will be 1.7.0)
  • updated Medic frankenportrait from @DuncanusDillius
  • even more validator fixes that I had thought were too difficult to do previously
  • start writing support conversations (in macros/conversations_support.cfg), for issue FE-style support conversations #73
  • improve S01 cutscene/dialogue
  • attempt at debugging odd bug where Lorin randomly disappears when visiting the gallows in S02
  • minor updates to the Akladian name generator

- create CODEOWNERS file
- enable optional advancements as per issue nemaara#118
- confirmation of @alxrem's translation credits entry
- start on ChangeLog
- updated Medic frankenportrait from @DuncanusDillius
- even more validator fixes that I thought were too difficult to do previously
- start writing support conversations
- improve S01 cutscene/dialogue
- attempt at debugging odd bug where Lorin randomly disappears when visiting the gallows in S02
- minor updates to the Akladian name generator
@cooljeanius cooljeanius requested a review from nemaara January 27, 2024 07:24
@cooljeanius cooljeanius self-assigned this Jan 27, 2024
@cooljeanius cooljeanius added the enhancement New feature or request label Jan 27, 2024
@cooljeanius cooljeanius added this to the 1.7.0 milestone Jan 27, 2024
@cooljeanius
Copy link
Collaborator Author

@nemaara any thoughts?

@nemaara
Copy link
Owner

nemaara commented Feb 18, 2024

it's 1000 lines so lemme comment tmrw or sth

@cooljeanius
Copy link
Collaborator Author

it's 1000 lines so lemme comment tmrw or sth

all of a sudden I've gotten all self-conscious about the support convos... feel free to tell me to leave those out if they make things too difficult to review...

remove unused Karen x Yahyazad S-rank convo
(I'd already ifdef-ed it out, anyways)
@cooljeanius
Copy link
Collaborator Author

it's 1000 lines so lemme comment tmrw or sth

all of a sudden I've gotten all self-conscious about the support convos... feel free to tell me to leave those out if they make things too difficult to review...

...ok, I trimmed them down a bit by removing one of them...

regenerate with `wmlxgettext` to reflect removal of unused conversation
@cooljeanius
Copy link
Collaborator Author

ok I got it down under 1000 lines with 481329d (most of the lines are in the potfile)

@nemaara
Copy link
Owner

nemaara commented Feb 26, 2024

I think some of the text could use revising but I don't have time to do that right now. I could add it in as is or would you like to try a second time at rewriting some lines? Basically I'm not sure some of the characters' personalities were captured in the newer text.

@cooljeanius
Copy link
Collaborator Author

cooljeanius commented Feb 28, 2024

I think some of the text could use revising but I don't have time to do that right now. I could add it in as is or would you like to try a second time at rewriting some lines? Basically I'm not sure some of the characters' personalities were captured in the newer text.

I mean, I guess another thing I could do would be to split the support convos off into a separate PR? Does everything else besides the support convos look fine?
Or, if it's just certain conversations that still need work, I could just split those off instead? How many of them should I be (temporarily) removing for improvement here?

Copy link
Owner

@nemaara nemaara left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Besides support convos the rest is fine.

scenarios/01_Breaking_the_Circle.cfg Show resolved Hide resolved
scenarios/01_Breaking_the_Circle.cfg Outdated Show resolved Hide resolved
cooljeanius and others added 2 commits April 1, 2024 23:54
Add additional comment to address point raised in PR
@cooljeanius cooljeanius merged commit 2ebbba3 into nemaara:master Apr 2, 2024
1 check passed
cooljeanius added a commit to cooljeanius/A_New_Order that referenced this pull request Apr 2, 2024
@cooljeanius cooljeanius deleted the start_writing_support_convos branch April 2, 2024 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable optional advancements?
2 participants