-
-
Notifications
You must be signed in to change notification settings - Fork 504
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
[DOCS]: Update wiki #4550
Comments
@cielf did you want to look at these patches? @jimmyli97 can you follow the approach mentioned here? https://stackoverflow.com/a/56810747/5199431 |
got it, I will add that to CONTRIBUTING as well the following should work to merge changes and clean up
|
I notice that we are still using diaper bank /diaperbank in several places. That should be changed to essentials bank. I don't see anything that I would block in these patches. |
Edited Home, BaseItem, and Application Overview to remove most instances of Diaperbank and replace with Essentials bank (left it alone in History sections, and also added a note that most common type is Diaperbank) Edits visible here |
I took a quick look and there are still some references to diaperbank on both Base Items and Application Overview . Also noting that It should be Essentials Bank , not Essentials bank. "essentials bank" is also ok, but the capitalized version is preferred. I don't know if that's what the pattern was before |
Capitalized all the Essentials Banks and removed all references to diaperbank except when talking about National Diaperbank Network on Application Overview. Edited Home, Application Overview, Base Items |
Although there are definitely still some old terminology uses in here (sb Product Drives instead of Diaper Drives, for instance), It's better than before, and I'm ok with it to go in. @jimmyli97 Is your goal for this PR to get the Wiki 100% up to date? |
@cielf I'm happy to do that, I just wanted to make onboarding easier by linking to an updated wiki in CONTRIBUTING however, I see a "Developer Architecture Overview" in |
I hadn't been giving the wiki any thought at all until you put in this PR. I don't think the two have to be together in the short to medium term, and it should be pretty easy to move as a separate effort later. |
In the longer term, it makes sense to me to have them together for ease of updating, but that can wait until the user guide is done, which will probably be the end of the year-ish. |
@cielf changed terminology to Product Drive, added DiaperDrive to history section, replaced Site Admin with Super Admin on Application Overview page I didn't notice anything else, let me know if there's anything else to change |
Seems good to me. |
Is there an existing request for this change?
Context for documentation change
Wiki is outdated:
Proposed solution
I forked it, created some commits, and created patches from those commits which someone with write access can approve and apply.
Preview of all changes at my fork
Patches available here:
Code of Conduct
The text was updated successfully, but these errors were encountered: