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.

Sign up for GitHub

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

Jump to bottom

Miscellaneous refactors, clean up #64

Merged
merged 21 commits into from
Jan 10, 2024
Merged

Miscellaneous refactors, clean up #64

merged 21 commits into from
Jan 10, 2024

Conversation

gangelo
Copy link
Owner

@gangelo gangelo commented Jan 8, 2024 •
edited
Loading

Miscellaneous refactors, clean up

@gangelo gangelo self-assigned this Jan 8, 2024
gangelo added 19 commits January 9, 2024 21:26
@gangelo
Correct error displayed if editor fails to open
796e0d4
@gangelo
Add ability to make screenshots for the dsu wiki easier
7f76145
@gangelo
Refactor code to exclude RawJsonFile class
8d6e3b4
@gangelo
Delete unneeded RawJsonFile class
341dd4b
@gangelo
Bump version and update gems
7ba0409
@gangelo
Update colorize gem
2135553
@gangelo
Start to refactor presenter logic
8ce9a7e
@gangelo
Remove unnecessary require
bbe24be
@gangelo
Use presenter for dsu add command display after adding an entry
6d76bc1
@gangelo
Use presenter to display browsed entry groups
e0b5570
@gangelo
Add star and share request in gemspec post install message
7715e82
@gangelo
Remove EntryGroupViewable module since it is replaced by the presenter
eae9abc
@gangelo
Raise an error if the display_nothing_to_list_message is called and t…
7d08053 
…here are entry groups to display
@gangelo
Add date_presenter specs
6244d10
@gangelo
Raise error if trying to call #display_nothing_to_list_message
ae462c1 
Because this method has no use for the DatePresenter because it will
always display an entry group.
@gangelo
Change the gemspec post_install_message
9cf4a29
@gangelo
Fix code to be more reliable
c2f98dd
@gangelo
Add DatesPresenter to list entry groups
4ee1d84
@gangelo
Add specs for DatePresenter and DatesPresenter
a96d949
@gangelo gangelo merged commit 0e9d4e5 into main Jan 10, 2024
4 checks passed
@gangelo gangelo deleted the development branch January 10, 2024 02:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@gangelo gangelo

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@gangelo