|
Hi Chris, I just updated my CP Vanity article, so I have new feedback on the process:
0.
Everything I said earlier still holds,
1.
except I did not notice any bug with respect to "new text + referenced files" (last paragraph in my previous message, now strikken). I modified three images, with new file names, and it worked as it should.
2.
As far as "delete files currently in use but no longer needed" my suggestion now is:
- let the user mark files he wants to remove;
- if they are not referenced at all, just delete them as you do now;
- if they are referenced, don't complain, keep a list; and try again when the wizard is about to close (no need to report success/failure IMO).
Luc Pattyn [Forum Guidelines] [My Articles] Nil Volentibus Arduum
Please use <PRE> tags for code snippets, they preserve indentation, improve readability, and make me actually look at the code.
|
|
|
|
|
There still is one more issue with the file attachments:
CP Vanity went from 2.2 to 2.3 earlier this week; it used to have a single zip (CPVanity.zip), the latest has two (CPVanity_exe.zip and CPVanity_src.zip). As I wasn't able to remove the first zip right away (it was still referenced by the old article text), I forgot to remove it in the end. No harm done, except for some disk space? Think again. The very nice "Browse Code" feature shows all the available zips, and as the old ZIP had quite a number of files, it filled my screen and I never considered scrolling down for more zips, so I was actually reading my own, now obsolete, 2.2 code.
The symptoms can be removed by going once more through the wizard and deleting the old zip; that is what I did today, and why CP Vanity now again features on the home page (while nothing was added or changed).
So I have to amend my earlier suggestion:
- on the upload page, take note which files need to be added (is OK) and are to be removed; if removal succeeds right away (because no reference exists), just remove them (or postpone, same difference); if removal fails, keep a list of things to remove.
- after the text has been finalized, scan the list of items to remove, and remove them if and only if not (or no longer) referenced.
- and this is the new part: also scan the article for things required and compare to the list of available files; if redundant files are present, launch a dialog and ask the user what to do about them, warning him that (a) they waste space, and (b) they may confuse the readers using "Browse Code". I am assuming things missing already are flagged (I never did that, so I don't know).
Luc Pattyn [My Articles] Nil Volentibus Arduum
The quality and detail of your question reflects on the effectiveness of the help you are likely to get. Please use <PRE> tags for code snippets, they improve readability. CP Vanity has been updated to V2.3
|
|
|
|
|
the plot thickens even more: I just discovered you don't have a (hidden) version control for the attachments, which means:
1.
if I remove graphics I no longer need, the older versions don't show themselves correctly any longer. You do check the references for the latest (or latest-but-one-to-be) version of the article, not older versions.
2.
if I reuse the filename for the ZIP file, and update the code, the older code is no longer available for browsing/downloading; even while viewing an old version of the article, what one gets when downloading is the latest file with the referenced name.
I was naively expecting you held copies of all files, one set for each version of the article. I can understand you may not be eager to do so.
Maybe it would be better for "Browse Code" to only show ZIP files that are referenced in the latest version of the article; that way authors could use different filenames for ZIP versions (supporting the download of code fitting an article version), without cluttering the "Browse Code" tab.
Luc Pattyn [My Articles] Nil Volentibus Arduum
The quality and detail of your question reflects on the effectiveness of the help you are likely to get. Please use <PRE> tags for code snippets, they improve readability. CP Vanity has been updated to V2.3
|
|
|
|
|
Thought it might be worth mentioning that the Quick Answers Bug List[^] and Suggestions/TODOs[^] has been updated and reworked a bit to allow easier tracking and reporting of fixes back to you guys.
Sorry for not being more consistent with this. Been away the last couple weeks and we've been working on some drawn out tasks. 
|
|
|
|
|
What happens if this big green button is pressed in Tips/Tricks?
I would assume that it would just change colour (much like accept answer in QA) and compliment the existing Tip but I want to make sure before I click it!
The mouseover tooltip doesn't really explain any implications.
Dave
Binging is like googling, it just feels dirtier. (Pete O'Hanlon)
BTW, in software, hope and pray is not a viable strategy. (Luc Pattyn)
Why are you using VB6? Do you hate yourself? (Christian Graus)
modified on Tuesday, March 23, 2010 5:33 PM
|
|
|
|
|
Yep, it basically says you approve of the alternate approach provided.
|
|
|
|
|
Thanks for the clarification - done it.
Dave
Binging is like googling, it just feels dirtier. (Pete O'Hanlon)
BTW, in software, hope and pray is not a viable strategy. (Luc Pattyn)
Why are you using VB6? Do you hate yourself? (Christian Graus)
|
|
|
|
|
It seems you can't delete a Quick Answer post after it has been edited by someone.
If the post has already been answered elsewhere, or otherwise inappropriate, it still should be able to be removed regardless how, how many times, it has been edited correct?
I know the language. I've read a book. - _Madmatt
|
|
|
|
|
Correct. I'm trying to think why exactly we put that restriction in place. I think it was just to prevent good/active content from being deleted. But.. that should be covered by the rating check.. Hrmmm...
Which answer are you specifically referring to? Just want to see an example. In any case, I'll add this to the list of TODOs (assuming I don't stumble across a good reason we put it there in the first place).
|
|
|
|
|
|
|
Looks like he simply re-asked the question in the Quick Answers system. I know that sounds like cheating a little but it doesn't hurt the system much. As long as it's not repeated within the same system. But your original points remains.
|
|
|
|
|
I would say its the same as posting to multiple forums. It looses the context of other answers and feedback that may already have been given.
I know the language. I've read a book. - _Madmatt
|
|
|
|
|
Mark Nischalke wrote: It looses the context of other answers and feedback that may already have been given.
True.
Mark Nischalke wrote: I would say its the same as posting to multiple forums.
I'd have to disagree. Some users will never use the forums for programming questions.
|
|
|
|
|
Thiru Thirunavukarasu wrote: I'm trying to think why exactly we put that restriction in place
That should be part of the specifications, the code comments, and the documentation. Are you hesitating which one to look in to?
WHY is the question that deserves most attention in documentation, it helps avoiding to make the same mistake over and over, as well as changing back and forth between alternatives.
|
|
|
|
|
Nope didn't hesitate to look at the code. Surprised we aren't perfect? It's sometimes hard to ensure every last piece of functionality is fully documented when dealing with large volumes of change requests.
|
|
|
|
|
If you enter the Jobs area via the Latest Jobs link, the FAQ & Pricing link is broken.
Sometimes you get a simple "Page Not Found" and others you get a "Page Not Found - The Most Likely Page your are looking for is....."
Update: Also the FAQ & Pricing link doesn't work from the dropdown menu either.
|
|
|
|
|
Thanks! This will be fixed in the next upload (today hopefully!)
cheers,
Chris Maunder
The Code Project | Co-founder
Microsoft C++ MVP
|
|
|
|
|
|
I keep seeing the following error
"-- Unable to load messages due to high load. Please try again --".
In fact, I've been getting it since morning - missed out on the CCC because of this error.
(not that I would have got the CCC anyway - but still....)
modified on Tuesday, March 23, 2010 10:20 AM
|
|
|
|
|
We are on it.
Sincerely,
Elina
Life is great!!!
Enjoy every moment of it! 
|
|
|
|
|
Elina Blank wrote: We are on it.
While you are fixing that, the "smiley" icons appear to have stopped wotking too....
*Edit - The "rant" icon is not working *
|
|
|
|
|
Abhinav S wrote: While you are fixing that, the "smiley" icons appear to have stopped wotking too....
I do see them.
Any special one is not working for you?
Sincerely,
Elina
Life is great!!!
Enjoy every moment of it!
|
|
|
|
|
It is back here
Sincerely,
Elina
Life is great!!!
Enjoy every moment of it!
|
|
|
|
|
cool - what was the issue ?
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
