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

Display New Version information from changelog at bottom of Info Page #240

Merged
merged 10 commits into from
Feb 1, 2021

Conversation

kdk1616
Copy link
Collaborator

@kdk1616 kdk1616 commented Jan 5, 2021

  • Parses Markdown from CHANGELOG.md and displays at bottom of info page

@kdk1616 kdk1616 requested review from psvenk, tektaxi and notrodes January 5, 2021 02:13
Copy link
Collaborator

@tektaxi tektaxi left a comment

Choose a reason for hiding this comment

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

Looks good, glad to be able to see updates in Aspine now.

@psvenk psvenk linked an issue Jan 5, 2021 that may be closed by this pull request
Copy link
Member

@psvenk psvenk left a comment

Choose a reason for hiding this comment

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

Looks good overall (from reading the diff), though I have not yet tested your code. Here are some code style changes (but take them with a grain of salt because I haven't tested any of my proposed changes).

public/home.html Outdated Show resolved Hide resolved
public/home.html Outdated Show resolved Hide resolved
public/js/home.js Outdated Show resolved Hide resolved
serve.js Outdated Show resolved Hide resolved
serve.js Outdated Show resolved Hide resolved
public/js/home.js Outdated Show resolved Hide resolved
public/js/home.js Outdated Show resolved Hide resolved
public/js/home.js Outdated Show resolved Hide resolved
public/js/home.js Outdated Show resolved Hide resolved
public/js/home.js Outdated Show resolved Hide resolved
@psvenk psvenk added this to the 2.7.0 milestone Jan 5, 2021
@notrodes
Copy link
Collaborator

Stop writing jquery!!

@notrodes
Copy link
Collaborator

Im fine not rewriting stuff, just stop writing morehow hard is that!?!?!?

Copy link
Collaborator

@notrodes notrodes left a comment

Choose a reason for hiding this comment

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

delete $

public/js/home.js Outdated Show resolved Hide resolved
public/js/home.js Outdated Show resolved Hide resolved
public/js/home.js Outdated Show resolved Hide resolved
public/css/home.css Outdated Show resolved Hide resolved
public/home.html Outdated Show resolved Hide resolved
@psvenk psvenk modified the milestones: 2.6.2, 2.7.0 Jan 25, 2021
kdk1616 and others added 8 commits January 31, 2021 20:56
Copy link
Member

@psvenk psvenk left a comment

Choose a reason for hiding this comment

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

Looks good to me; I think this is ready to merge pending @notrodes' approval. I noticed that the changelog inclusion does not work in Aspine Lite, but I chose to split that into a separate issue (#250) (a) this is a sufficiently small feature, (b) the link to GitHub still exists on Aspine Lite, and (c) implementing this would require considerable work on the build script.

@psvenk psvenk requested a review from notrodes February 1, 2021 02:08
To improve performance, render the changelog when starting the server
instead of doing it whenever the endpoint is reached.
@psvenk psvenk dismissed notrodes’s stale review February 1, 2021 03:00

jQuery has been removed.

@psvenk psvenk merged commit 9e0915b into master Feb 1, 2021
@psvenk psvenk deleted the changelog branch February 1, 2021 03:00
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.

Include a way to view the changelog in Aspine
4 participants