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

[Fix] #6 #33

Closed
wants to merge 1 commit into from
Closed

[Fix] #6 #33

wants to merge 1 commit into from

Conversation

Kristinita
Copy link

@Kristinita Kristinita commented Apr 5, 2017

1. Behavior before pull request

I create a new Markdown file → Ctrl+Shift+P (⌘⇧p for Mac) → Pelican: Insert Metadata → metadata insert:

Date Metadata

Ctrl+Shift+P (⌘⇧p for Mac) → Pelican: Update Article DateDate metadata variable update:

Update Date

2. Behavior after pull request

I add a new metadata variable Modified.

I create a new Markdown file → Ctrl+Shift+P (⌘⇧p for Mac) → Pelican: Insert Metadata → metadata with new Modified variable insert, Modified value = Date value:

Modified metadata

Ctrl+Shift+P (⌘⇧p for Mac) → Pelican: Update ModifiedModified variable update, Date variable stay constantly.

Update Modified

3. Argumentation

See @jachin arguments. Date variable — time, when we create article or page. Modified variable — last update time article or page. We must update Modified, not Date. See File Metadata section in official documentation.

4. Testing environment

Operating system and version:
Windows 10 Enterprise LTSB 64-bit EN
Sublime Text:
Build 3126
Syntax:
Markdown

5. Notes

  1. I'm sorry, I don't know reStructuredText — I don't know, how my changes will work in rst files.
  2. This pull request don't solve [Feature request] Modified setting update on save #30 issue. Modified variable doesn't update by save.

Thanks.

@Kristinita
Copy link
Author

Kristinita commented Apr 5, 2017

Sorry, I don't see, that for Sublime Text 3 you have a specific branch.

@Kristinita Kristinita closed this Apr 5, 2017
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