Skip to content

Conversation

@seyfeb
Copy link
Collaborator

@seyfeb seyfeb commented Nov 9, 2020

Regarding #131, I added schema.org Recipe’s dateCreated and dateModified fields to the recipe view.

These also set/updated when creating or updating a recipe.

The recipe view

  • shows the created date if available
  • shows the modified date if its
    • available
    • not the same as the dateCreated timestamp

Date & time are localized.

Screenshot 2020-11-09 at 14 38 48

@christianlupus christianlupus linked an issue Nov 12, 2020 that may be closed by this pull request
@seyfeb seyfeb force-pushed the feature/addDateCreated branch from 2458c5f to bafbf87 Compare November 14, 2020 17:32
@christianlupus christianlupus added the Frontend Issue or PR related to the frontend code label Nov 23, 2020
Copy link
Collaborator

@christianlupus christianlupus left a comment

Choose a reason for hiding this comment

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

The static code analysis looks good. I would like to try it in a test instance then I will approve it.

@christianlupus christianlupus added the Backend Issue or PR related to the backend code label Nov 23, 2020
@codecov
Copy link

codecov bot commented Dec 6, 2020

Codecov Report

Merging #377 (a85893d) into master (7c89ba9) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##             master    #377      +/-   ##
===========================================
- Coverage      0.92%   0.92%   -0.01%     
- Complexity      410     412       +2     
===========================================
  Files            13      13              
  Lines          1300    1304       +4     
===========================================
  Hits             12      12              
- Misses         1288    1292       +4     
Flag Coverage Δ Complexity Δ
integration 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
unittests 0.92% <0.00%> (-0.01%) 0.00 <0.00> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
lib/Service/RecipeService.php 0.00% <0.00%> (ø) 235.00 <0.00> (+2.00)

@christianlupus christianlupus merged commit f9bfc95 into nextcloud:master Dec 21, 2020
@seyfeb seyfeb deleted the feature/addDateCreated branch December 30, 2020 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backend Issue or PR related to the backend code Frontend Issue or PR related to the frontend code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support "dateCreated" field

2 participants