-
Notifications
You must be signed in to change notification settings - Fork 42
Release 8.1.0 #300
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
Merged
Merged
Release 8.1.0 #300
Changes from all commits
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
183757f
Issue #ED-619 feat: Update dependencies and use dayjs instead of mome…
Ajoymaity bdaba59
Issue #ED-619 fix: Updated package version
Ajoymaity 8103f67
Issue #ED-619 fix: Update dependencies
Ajoymaity b18a0ca
Issue #ED-619 merge: Merge pull request #259 from Ajoymaity/release-5…
swayangjit 33cff76
Issue #0000 fix: Updated the cs-telemetry sevices for search
gouravmore e85f168
Issue #0000 fix: Updated the cs-telemetry sevices for search
gouravmore 52b549e
Issue #0000 merge Merge pull request #262 from gouravmore/release-5.1.1
swayangjit f602c30
Issue #ED-0000 fix: Updated typescript version and fix on typescript …
vpPavithra 88586b7
Issue #ED-0000 fix: Updated test cases
vpPavithra 7001846
Issue #ED-1950 fix: Updated webpack script
vpPavithra ec97360
Issue #ED-1950 fix: webpack fix
vpPavithra a0b7c2d
Issue #ED-1950 merge: Merge pull request #263 from vpPavithra/releas…
swayangjit 3c1e67a
Issue #ED-2131 fix: sdk local link fix
vpPavithra a97117c
Issue #ED-2131 fix: sdk local link fix
vpPavithra cf52f69
Issue #ED-2131 fix: updated package-lock on merge conflicts
vpPavithra 6885d0b
Issue #ED-2131 test: test case fix
vpPavithra 2ce9c87
Issue #ED-2131 fix: script fix
vpPavithra 44cb79f
Issue #ED-2131 fix: fix on ts-loader on publish npm
vpPavithra 58660fd
Issue #ED-2131 merge: Merge pull request #266 from vpPavithra/release…
swayangjit 9c21db9
Issue #ED-2855 feat: Removed hardcoded of diksha ncert and igot
Ajoymaity 5fcf40d
Issue #SB-ED-3073 feat: Added framework config implementation in fram…
swayangjit fa4dd3a
Issue #SB-ED-3073 merge: Merge pull request #268 from swayangjit/rel…
swayangjit c03d925
Issue #SB-ED-3073 chore: Updated package version.
swayangjit 0cfe271
Issue #SB-ED-3073 merge: Merge pull request #269 from swayangjit/rel…
swayangjit 4fa54f9
Issue #SB-ED-3073 fix: Updated getCategoryMap issue.
swayangjit f2035d9
Issue #SB-ED-3073 merge: Merge pull request #270 from swayangjit/rel…
swayangjit 94a2fe6
Issue #SB-ED-3073 chore: Updated index in getFrameworkConfig.
swayangjit 97fa719
Issue #SB-ED-3073 merge: Merge pull request #271 from swayangjit/rel…
swayangjit 31d95ed
Issue #SB-ED-3073 chore: Updated requiredCategories attribute.
swayangjit 5a5c2e0
Issue #SB-ED-3073 merge: Merge pull request #272 from swayangjit/rele…
swayangjit f4b12b5
fix: package.json & package-lock.json to reduce vulnerabilities
snyk-bot 5476414
Merge branch 'release-7.0.0' into snyk-fix-1a0f156a2bb82635cab5e2f1fd…
vinukumar-vs db0c553
Merge pull request #6 from vinukumar-vs/snyk-fix-1a0f156a2bb82635cab5…
vinukumar-vs e80122e
Merge branch 'release-7.0.0' of https://github.com/vinukumar-vs/sunbi…
vinukumar-vs 150fe58
Issue #ED-3183 fix: Vulnerabilities fixes of Telemetry SDK & JSON-LD
vinukumar-vs 8c7c69e
Issue #ED-3183 fix: Vulnerabilities fixes suggested by Snyk
vinukumar-vs e42c134
Issue #SB-ED-3073 chore: Added name in the response.
swayangjit e133f41
Issue #SB-000 merge: Merge branch 'release-7.0.0' of github.com:Sunbi…
swayangjit 4b71c9c
Issue #SB-000 merge: Merge branch 'release-7.0.0' of github.com:Sunbi…
swayangjit 6687ae0
Merge pull request #273 from swayangjit/release-7.0.0
rajeevsathish decf2a1
Issue #ED-3658 fix:Updated telemetrysdk to latest version
sujithsojan edf9cec
Issue #ED-3658 fix: Updated the package-lock
sujithsojan 2097913
Issue #ED-3658 fix: Updated the package versions
sujithsojan 4904d9e
Issue #ED-3658 merge: Merge pull request #274 from sujithsojan/releas…
swayangjit b66fcc6
Issue #ED-3659 feat:Vulnarability fixes for sdk repo
sujithsojan a1ab509
Issue #ED-3659 merge: Merge pull request #275 from sujithsojan/releas…
swayangjit a6c10b4
ED-4039 fix:updated package for snyk Vulnerabilities
princegupta1131 29c212b
ED-4039 fix:updated package for snyk Vulnerabilities
princegupta1131 231dc21
ED-4039 fix:updated package for snyk Vulnerabilities
princegupta1131 cdf6914
ED-4022 fix: circle cli issue
princegupta1131 e90d569
Merge branch 'release-8.0.0' into release-7.0.0
rajeevsathish 5a2cf43
Merge pull request #267 from Ajoymaity/release-7.0.0
rajeevsathish a895491
Merge branch 'release-8.0.0' into 8.0.0_snyk_
rajeevsathish 592fcb9
Merge pull request #277 from princegupta1131/8.0.0_snyk_
rajeevsathish 325a824
testing pull request yml file
Deeksha1502 5e73928
modified webpack
Deeksha1502 44f00ef
SBCOSS-473: Added GitHub actions for raising pull request
Deeksha1502 5c4be87
SBCOSS-473: Removed config yml file
Deeksha1502 750018c
Merge pull request #284 from Deeksha1502/pull-request-yml
HarishGangula 6d5b7a0
SBCOSS-474: Added GitHib actions for publishing to NPM
Deeksha1502 65b88ce
SBCOSS-474: Updated the readme with the latest changes
Deeksha1502 f0f03fa
Merge pull request #286 from Deeksha1502/publish-sdk
HarishGangula ff22b8b
SBCOSS-557: updated bmgs in interfaces
Rakshitha-D 02805d4
SBCOSS-557: updated package version
Rakshitha-D 2b11244
Merge branch 'release-8.0.0' into bmgsFix
Rakshitha-D d1c874c
SBCOSS-557: updated group generator
Rakshitha-D 1ae45bc
Merge branch 'bmgsFix' of https://github.com/Rakshitha-D/sunbird-clie…
Rakshitha-D 5be78e6
SBCOSS-557: updateto enhance type safety
Rakshitha-D 318f058
Merge pull request #289 from Rakshitha-D/bmgsFix
manjudr 9cced02
SBCOSS-557: resolved jsonld global object dependency issue
Rakshitha-D 66ddecf
Merge pull request #292 from Rakshitha-D/bmgsFix
manjudr 18f807d
SB-I367: cryptold version change to support RSAKeyPair constructor.
JeraldJF 7664b00
Merge pull request #294 from JeraldJF/release-8.0.0
pallakartheekreddy a004ad3
SB-I367: generate fresh lock file on installation
JeraldJF 50ce8f7
Merge pull request #295 from JeraldJF/release-8.0.0
pallakartheekreddy 4dc143a
SB-I367: version of jsonld signatures reverted to v7
JeraldJF aab3b64
Merge pull request #296 from JeraldJF/release-8.0.0
pallakartheekreddy 31ac58b
Bump version from 8.0.3 to 8.0.4
pallakartheekreddy 11c524e
Bump version to 8.0.5 and update jsonld dependency
pallakartheekreddy 312f2f6
SB-I367: jsold versiosn changes for 8.1.0 version
JeraldJF 9a5f6c7
Merge pull request #298 from JeraldJF/release-8.1.0-cs
pallakartheekreddy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,44 @@ | ||
| name: Create Tag on Push | ||
|
|
||
| on: | ||
| push: | ||
| tags: | ||
| - '*' | ||
|
|
||
| jobs: | ||
| publish-npm: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Set up Node.js | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: '22.15' | ||
|
|
||
| - name: Check installed versions | ||
| run: | | ||
| node -v | ||
|
|
||
| - name: Install dependencies | ||
| run: | | ||
| npm install | ||
|
|
||
| - name: Install webpack | ||
| run: npm install --save-dev webpack webpack-cli | ||
|
|
||
| - name: Build and Pack | ||
| run: | | ||
| npm run build:prod | ||
| npm pack ./dist | ||
|
|
||
| - name: Publish to NPM | ||
| env: | ||
| NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }} | ||
| run: | | ||
| echo "//registry.npmjs.org/:_authToken=${NODE_AUTH_TOKEN}" > ~/.npmrc | ||
| npm publish project-sunbird-client-services-* | ||
|
|
||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,61 @@ | ||
| name: PR Build | ||
|
|
||
| on: | ||
| pull_request: | ||
|
|
||
| jobs: | ||
| build-pr: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Set up Node.js | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: '22.15' | ||
|
|
||
| - name: Check installed versions | ||
| run: | | ||
| node -v | ||
|
|
||
| - name: Restore node_modules cache | ||
| id: cache | ||
| uses: actions/cache@v3 | ||
| with: | ||
| path: node_modules | ||
| key: ${{ runner.os }}-node-modules-${{ hashFiles('**/package-lock.json') }} | ||
| restore-keys: | | ||
| ${{ runner.os }}-node-modules- | ||
|
|
||
| - name: Install dependencies | ||
| run: | | ||
| npm i | ||
|
|
||
| - name: Save node_modules cache | ||
| if: steps.cache.outputs.cache-hit != 'true' | ||
| uses: actions/cache@v3 | ||
| with: | ||
| path: node_modules | ||
| key: ${{ runner.os }}-node-modules-${{ hashFiles('**/package-lock.json') }} | ||
|
Rakshitha-D marked this conversation as resolved.
|
||
|
|
||
| - name: Install webpack | ||
| run: npm install --save-dev webpack webpack-cli | ||
|
|
||
| - name: Install SonarQube scanner | ||
| run: npm install -g sonarqube-scanner | ||
|
|
||
| - name: Run SonarScanner | ||
| run: sonar-scanner | ||
|
Rakshitha-D marked this conversation as resolved.
|
||
|
|
||
| - name: Build project and Pack (PR) | ||
| run: | | ||
| npm run build:prod | ||
| npm pack ./dist | ||
|
|
||
| - name: Run lint | ||
| run: npm run lint | ||
|
Rakshitha-D marked this conversation as resolved.
|
||
|
|
||
| - name: Run tests with coverage | ||
| run: npm run test:ci | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,6 +2,8 @@ | |
|
|
||
| ## Table of Contents | ||
|
|
||
| 1. [Development](#development-setup) | ||
| 1. [Continuous Integration](#continuous-integration) | ||
| 1. [Overview](#overview) | ||
| 1. [Installation](#installation) | ||
| 1. [Getting Started](#getting-started) | ||
|
|
@@ -10,6 +12,73 @@ | |
| 1. [Services](#services) | ||
| 1. [CsGroupService](#csgroupservice) | ||
|
|
||
| ## Development setup | ||
|
|
||
| ### Prerequisites | ||
|
|
||
| NodeJS 22.15 | ||
|
|
||
| ### Local Development | ||
|
|
||
| 1. Fork the repository on GitHub | ||
|
|
||
| 2. Clone your fork: | ||
|
|
||
| ``` | ||
| git clone https://github.com/your-username/sunbird-client-service.git | ||
|
|
||
| cd sunbird-client-service | ||
| ``` | ||
| 3. Install dependencies: | ||
|
|
||
| ``` | ||
| npm i | ||
| ``` | ||
| 4. To run the project | ||
| ``` | ||
| npm run build | ||
| ``` | ||
| ## Continuous Integration | ||
|
|
||
| This project uses GitHub Actions for automated testing, building, and publishing. Below are details of the CI processes: | ||
|
|
||
| ### Pull Request Workflow | ||
|
|
||
| When a pull request is opened or updated, the following automated checks are performed: | ||
|
|
||
| 1. **Code checkout**: The repository code is checked out | ||
| 2. **Environment setup**: Node.js 22.15 is installed and configured | ||
| 4. **Test execution**: All tests are run with coverage reporting | ||
| 5. **Code linting**: ESLint is run to check code style and quality | ||
| 6. **Static code analysis**: SonarQube scan is performed to evaluate code quality | ||
| 6. **Build verification**: The project is built using webpack to ensure it compiles correctly. Ensure ```SONAR_TOKEN``` is configured in your repository. | ||
|
|
||
|
Comment on lines
+51
to
+55
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix duplicate step numbering in PR Workflow section. The numbered list has duplicate "6" entries at lines 54-55. The numbering should be sequential (4, 5, 6, 7). Proposed fix 1. **Code checkout**: The repository code is checked out
2. **Environment setup**: Node.js 22.15 is installed and configured
-4. **Test execution**: All tests are run with coverage reporting
-5. **Code linting**: ESLint is run to check code style and quality
-6. **Static code analysis**: SonarQube scan is performed to evaluate code quality
-6. **Build verification**: The project is built using webpack to ensure it compiles correctly. Ensure ```SONAR_TOKEN``` is configured in your repository.
+3. **Test execution**: All tests are run with coverage reporting
+4. **Code linting**: ESLint is run to check code style and quality
+5. **Static code analysis**: SonarQube scan is performed to evaluate code quality
+6. **Build verification**: The project is built using webpack to ensure it compiles correctly. Ensure ```SONAR_TOKEN``` is configured in your repository.🤖 Prompt for AI Agents |
||
| This ensures that all changes in pull requests meet quality standards before merging. | ||
|
|
||
| ### Release Workflow | ||
|
|
||
| When a new tag is pushed to the repository, the following automated steps are executed: | ||
|
|
||
| 1. **Code checkout**: The tagged version of code is checked out | ||
| 2. **Environment setup**: Node.js 22.15 is installed and configured with NPM registry access | ||
| 3. **Dependency installation**: Project dependencies are installed via `npm i` | ||
| 4. **Build process**: The project is built for production | ||
| 5. **Package creation**: The build output is packaged for distribution | ||
| 6. **NPM publication**: The package is published to the NPM registry after setting ```NPM_TOKEN``` | ||
|
|
||
| This allows for simple, reliable releases by simply pushing a new tag. | ||
|
|
||
| ### Manual Release Process | ||
|
|
||
| To manually trigger a new release: | ||
|
|
||
| 1. Update the version in `package.json` | ||
| 2. Commit the change: `git commit -m "Increase version to x.y.z"` | ||
| 3. Create and push a tag: `git tag v{x.y.z} && git push origin v{x.y.z}` | ||
|
|
||
| The GitHub Actions workflow will automatically build and publish the new version. | ||
|
|
||
|
|
||
| ## Overview | ||
|
|
||
| The library is grouped into Modules and SubModules as shown below - | ||
|
|
@@ -258,4 +327,4 @@ CsModule.instance.init({ | |
| export interface CsCourseService { | ||
| getUserEnrollmentList(request: GetUserEnrollmentListRequests, additionalParams?: { [key: string]: string }, config?: CsCourseServiceConfig): Observable<Course[]>; | ||
| } | ||
| ``` | ||
| ``` | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.