Skip to content

Commit 75754a7

Browse files
committed
wip
1 parent 02f3149 commit 75754a7

File tree

12 files changed

+30
-37
lines changed

12 files changed

+30
-37
lines changed

.github/CONTRIBUTING.md

+4-4
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
1-
# VueCrudX Contributing Guide
2-
Hello and thank you for your interest in helping make VueCrudX better. Please take a few moments to review the following guidelines:
1+
# cookbook Contributing Guide
2+
Hello and thank you for your interest in helping make cookbook better. Please take a few moments to review the following guidelines:
33

44
## IMPORTANT INFORMATION
5-
* For general questions, please join [TBD - our Discord Community](https://community.TBD.com/).
5+
* For general questions, please join [Our Discussion Board](https://github.com/ais-one/cookbook/discussions).
66

77
## Reporting Issues
88
* The issue list of this repo is **exclusively** for Bug Reports and Feature Requests.
99
* Bug reproductions should be as **concise** as possible.
1010
* **Search** for your issue, it _may_ have been answered.
1111
* See if the error is **reproduceable** with the latest version.
12-
* If reproduceable, please provide a [Codepen](https://template.tbd.com) or public repository that can be cloned to produce the expected behavior. It is preferred that you create an initial commit with no changes first, then another one that will cause the issue.
12+
* If reproduceable, please provide a [Codepen](https://codepen.io/) or public repository that can be cloned to produce the expected behavior. It is preferred that you create an initial commit with no changes first, then another one that will cause the issue.
1313
* **Never** comment "+1" or "me too!" on issues without leaving additional information, use the :+1: button in the top right instead.
1414

1515
## Pull Requests

.github/PULL_REQUEST_TEMPLATE.md

+1-4
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
11
### Please read and mark the following check list before creating a pull request (check one with "x"):
22

3-
- [ ] I read and followed the [Contribution guide](https://github.com/ais-one/cookbook/blob/master/.github/CONTRIBUTING.md)
4-
<!-- TBD
5-
- [ ] I read and followed the [New Feature Checklist](https://github.com/ais-one/cookbook/blob/master/.github/DEV_DOCS.md#new-feature-checklist)
6-
-->
3+
- [ ] I read and followed the [Contribution guide](https://github.com/ais-one/cookbook/blob/main/.github/CONTRIBUTING.md)
74

85
#### Short description of what this resolves:

CHANGELOG.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,7 @@
250250
- set our internal usage key to __key, to avoid collisions
251251
- hide = hide table
252252
- filter & sort is not enabled by default, need to set as true
253-
- [common-lib/esm/bwc-t4t-form.js] - (generic crud) create form using web components - currently only input tag handled (TBD add select tag, and other custom tags, handle file inputs)
253+
- [common-lib/esm/bwc-t4t-form.js] - (generic crud) create form using web components - currently only input tag handled (TODO add select tag, and other custom tags, handle file inputs)
254254
- [common-lib/esm/t4t-fe.js] - (generic crud) created utilities also includes validation...
255255
- [common-lib/esm/t4t-validate.js] - (generic crud) created validation for backend use
256256
- [example-vite] NOTE: mwc-multiselect.js is broken

README.md

+15-19
Original file line numberDiff line numberDiff line change
@@ -6,42 +6,38 @@
66
[![Known Vulnerabilities](https://snyk.io/test/github/ais-one/cookbook/badge.svg)](https://snyk.io/test/github/ais-one/cookbook)
77
[![MadeWithVueJs.com shield](https://madewithvuejs.com/storage/repo-shields/823-shield.svg)](https://madewithvuejs.com/p/cookbook/shield-link)
88

9-
## IMPORTANT - Read Me First!
9+
### 1 - IMPORTANT - Read Me First!
1010

11-
### 1 - Updateable Templates
11+
### 1.1 - Updateable Templates
1212

1313
Your project is created using a template. If template has updates, can the upstream changes be managed with minimal impact on userland codes?
1414

15-
Yes and our project achieve this through:
16-
15+
Yes and this is achieved through:
1716
- Design
18-
- Exclude configurable files from repo (e.g. .env), but include sample (e.g. .env.sample) for reference
1917
- Create folder where all userland code is placed, template must NOT touch this folder
18+
- template should not to be part of a monorepo
2019
- Process
21-
- clone template repo
22-
- create a remote called `upstream` pointing to template repo
20+
- clone template and create remote called `upstream` pointing to template
2321
- update framework when necessary by merging `upstream` into `origin`
2422

25-
Updateable template should not to be part of a monorepo
26-
27-
### 2 - Manageable Sharing
23+
### 1.2 - Manageable Sharing
2824

2925
You have code shared between multiple projects and libraries. If the code is updated. Can you avoid breaking your dependents and dependencies?
3026

31-
Yes, by basing on the following principles
27+
Yes, based on the following principles:
3228
- Shared libraries should be isolated and versioned, dependents can stay on last-known-good version and update when ready
3329
- Isolation and versioning can be extended to `types` (for Typescript) and `contracts` (for API)
3430
- minimize inter & nested dependencies, and technical debt
3531

3632
---
3733

38-
### 3 - General Requirements
34+
### 2 - General Requirements
3935

4036
- git, github (for actions, secrets, etc) & IDE (e.g. vscode), Docker
41-
- use unix shell (Windows use git-bash or WSL2)
37+
- unix shell (Windows use git-bash or WSL2)
4238
- node 18+ LTS & npm 9+ (npm i -g npm@latest `to update`)
4339

44-
### 4 - Recipies
40+
### 3 - Recipies
4541

4642
[Projects](recipies/README.md) that can be used (express and vuejs template, shareable libraries and tools)
4743

@@ -77,19 +73,19 @@ Yes, by basing on the following principles
7773

7874
---
7975

80-
### 5 - Sandbox
76+
### 4 - Sandbox
8177

8278
Research and exploration [projects](sandbox/README.md)
8379

84-
### 6 - Docker Dev Env
80+
### 5 - Docker Dev Env
8581

8682
Container setups for supporting apps for local development and testing [docker-devenv/README.md]()
8783

88-
### 7 - Documentation
84+
### 6 - Documentation
8985

90-
The [docs](docs/home.md) folder contains documentation on other related/unrelated topics and is in the midst of a major update
86+
The [docs](docs/home.md) folder contains useful information is in the midst of a major cleanup
9187

92-
### 8 - Useful scripts - For Use By Maintainers
88+
### 7 - Useful scripts - For Use By Maintainers
9389

9490
- `bulk-git.sh`: script to diff, pull, push git (for repos in `recipies`)
9591
- `bulk-npm.sh`: script to check for and/or update dependencies in package.json (for repos in `recipies`)

docs/cloud.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
Where GCP, AWS, Azure Aliyun, Most concepts are the same
44

5-
Try to use terraform (TBD refer to DSO project)
5+
Try to use terraform (TODO refer to DSO project)
66

77
## Install CLI
88

docs/deployment.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -433,7 +433,7 @@ docker exec -it <container id> /bin/bash
433433
# to save an image
434434
docker save image:tag | gzip > image-tag.tar.gz
435435

436-
# TBD load an image from tar.gz
436+
# TODO load an image from tar.gz
437437
```
438438

439439
## Docker Commands

docs/js.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ Bind is different. It always returns a new function. We can use Bind to curry fu
138138
139139
### Classes / Prototypes
140140
141-
TBD
141+
TODO
142142
143143
## Scope
144144

docs/nodejs.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -183,4 +183,4 @@ You need to create `tag` on the github remote also by creating a new release or
183183

184184
- https://blog.logrocket.com/optimize-node-js-performance-with-clustering/
185185
- https://blog.logrocket.com/customizing-node-js-env-files/
186-
- TBD workerthreads
186+
- TODO workerthreads

docs/sqa-nodejs.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -108,11 +108,11 @@ pre-commit sample file is created in .husky/pre-commit
108108

109109
## commitizen
110110

111-
TBD
111+
TODO
112112

113113
## semantic version
114114

115-
TBD
115+
TODO
116116

117117
# A list of Software Quality Assurance Tools
118118

recipies/README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,6 @@ The packages in the `libs` workspace are published to npm. They are shared, vers
2222
The packages in `tools` workspace contain the following useful sample applications
2323
- dbdeploy: for DB migration and seeding
2424

25-
### 4 [ReactJS Template - TBD]()
25+
### 4 [ReactJS Template - TODO]()
2626

2727

sandbox/aws/index.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ export const readFile = async (keyName: string | undefined) => {
7575
Key: keyName,
7676
});
7777

78-
// TBD try and promisify
78+
// TODO try and promisify
7979
try {
8080
const response = await client.send(command);
8181
const input: SdkStream<any> | undefined = response.Body

sandbox/services/README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ See output when sending message from a client (currently client that sent will a
7171

7272
If going through a proxy such as load balancer, need sticky sessions
7373

74-
TBD: handle duplicate message to message originator client on subscription message
74+
TODO: handle duplicate message to message originator client on subscription message
7575

7676

7777
---

0 commit comments

Comments
 (0)