question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Misleading notification when changing file type via JekyllAdmin

See original GitHub issue

Describe in one or two sentences why you’re opening this issue:

Renaming a .txt file as .md file throws a misleading flash notification

Tell us a bit about yourself

  • Version of JekyllAdmin I’m using <hint: use bundle show to check>: 0.1.0
  • Version of Jekyll I’m using <hint: use bundle show to check>: 3.2.1
  • Operating System <e.g. OS X, Windows>: Windows 7
  • Browser <e.g, Safari, Chrome>: Chrome

Steps to reproduce

Navigate to http://localhost:3000/admin/page/new create demo.txt by entering that in the breadcrumb rename demo.txt to demo.md in the breadcrumb

Expected

Expected demo.txt to be renamed to demo.md silently or with a success! notification

Actual

demo.txt is successfully renamed to demo.md but with a error flash notification (couldn’t update page…)

Other details

  • This could be a false alarm as the plugin is infact rightfully stating that demo.txt couldn’t be updated as its no longer available.
  • Its nevertheless an alarm that may cause confusion for a short while. Possible solution: Introduce a flash notification to handle renaming. This will return a message echoing the old filename with extn and the new filename with extn

Issue Analytics

  • State:closed
  • Created 7 years ago
  • Comments:5 (5 by maintainers)

github_iconTop GitHub Comments

1reaction
mertkahyaoglucommented, Feb 22, 2017

Closed via #307

0reactions
ashmarolicommented, Jan 6, 2017

We should either not allow creating pages with .txt extension

I think it would be better to not allow change of file extension / type via the Admin interface.

or not add any front matter if the extension is not markdown

don’t think that’s a good idea. Even *.html with Front Matter is a valid JekyllPage object.

Read more comments on GitHub >

github_iconTop Results From Across the Web

PageCreate can create .txt file but treats it like a .md file #204
Version of JekyllAdmin I'm using <hint: use bundle show to check>: ... Misleading notification when changing file type via JekyllAdmin #205.
Read more >
Jekyll-admin fails with NoMethodError - undefined method ...
The issue is caused by the presence of a config file in the theme-gem you're using. Therefore instruct Jekyll to ignore the theme-config...
Read more >
Jekyll Admin - Containers
Main container for metafields. Generates list, object or plain inputs for front matters other than title , body , path and draft ....
Read more >
Use Jekyll Manager Instead of Jekyll Admin | FlutterGeek
I no longer have to drag and drop using Finder for this. The metadata too has changed. It is, by default, hidden and...
Read more >
Get notified when members of your team update your shared file
In App - You can receive notifications right within Word, PowerPoint, or Excel, when people you've shared your file with make changes, edits,...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found