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

🥗 Journal: Test adding an Entry#summary when Writing Entries #9

Merged

Conversation

zspencer
Copy link
Member

@zspencer zspencer commented Jan 29, 2024

This is a quick line-of-action end-to-end test for setting an Entry#summary.

zspencer added a commit to zinc-collective/convene that referenced this pull request Jan 29, 2024
- zinc-collective#8
- zinc-collective#10

When I started on this, I started with [changes to the data
layer], because that's where my brain tends to naturally start.

But when I wrote the [system test], I realized it was more of an
[`Entries#summary`].

So I am renaming from `Entries#description` to `Entries#summary`.

Thank you for coming to my TED Talk.

[`Entries#summary`]: zinc-collective#9
[system test]: zinc-collective#9
[changes to the data layer]: zinc-collective#8
zspencer added a commit that referenced this pull request Jan 29, 2024
- #8
- #10

When I started on this, I started with [changes to the data
layer], because that's where my brain tends to naturally start.

But when I wrote the [system test], I realized it was more of an
[`Entries#summary`].

So I am renaming from `Entries#description` to `Entries#summary`.

Thank you for coming to my TED Talk.

[`Entries#summary`]: #9
[system test]: #9
[changes to the data layer]: #8
@zspencer zspencer force-pushed the journal/entry/system-spec-for-summary branch 2 times, most recently from 8f24440 to 56f0f2a Compare January 29, 2024 04:37
Base automatically changed from journal/entry/short-description to journal/entry-summaries January 29, 2024 04:40
@zspencer zspencer force-pushed the journal/entry/system-spec-for-summary branch from 56f0f2a to fdb42e8 Compare January 29, 2024 04:42
@zspencer zspencer changed the title 🥗 Journal: Writing Entries: Test adding Entry#summary 🥗 Journal: Test adding an Entry#summary when Writing Entries Jan 29, 2024
@zspencer zspencer force-pushed the journal/entry/system-spec-for-summary branch 2 times, most recently from d703b48 to 507fe96 Compare January 29, 2024 05:02
- #10
- #2

This is a quick line-of-action end-to-end test for setting an `Entry#summary`.
@zspencer zspencer force-pushed the journal/entry/system-spec-for-summary branch from 507fe96 to 0ac0ba5 Compare January 29, 2024 20:47
@zspencer zspencer merged commit f593fad into journal/entry-summaries Jan 29, 2024
2 of 3 checks passed
@zspencer zspencer deleted the journal/entry/system-spec-for-summary branch January 29, 2024 20:52
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.

1 participant