Skip to content
This repository was archived by the owner on Apr 15, 2024. It is now read-only.

Message when rows are limited#604

Merged
yreddy-CS merged 183 commits intodevelopfrom
tasks-err-log
Jul 14, 2022
Merged

Message when rows are limited#604
yreddy-CS merged 183 commits intodevelopfrom
tasks-err-log

Conversation

@yreddy-CS
Copy link
Copy Markdown
Collaborator

@yreddy-CS yreddy-CS commented Jul 2, 2022

Checklist:

Please check you fulfill ALL of the relevant checkboxes

  • Notified docs of any potential USER-facing changes
  • Added short description of the change - with relevant motivation and context.
  • Branch has the ticket number in its name (along with a ticket summary)
  • Commented the code, particularly in hard-to-understand areas
  • Added tests that prove my fix is effective or that my feature works
  • Ran npm test and it passes
  • Changes generate no new warnings
  • Changes are backward compatible with older clusters

dschueller-cs and others added 30 commits November 5, 2021 12:49
* workspace command now creates skill templates fromn the cortex-code repository.
# Conflicts:
#	package-lock.json
#	src/commands/utils.js
* Added new config version 4, which includes template repo config and IDE registry support.
* Removed old generate templates
* Build command works
* Started on publish command (incomplete)
# Conflicts:
#	package-lock.json
#	package.json
#	src/commands/utils.js
# Conflicts:
#	package-lock.json
#	package.json
yreddy-CS and others added 18 commits June 21, 2022 13:38
Bumps [got](https://github.com/sindresorhus/got) from 11.8.3 to 11.8.5.
- [Release notes](https://github.com/sindresorhus/got/releases)
- [Commits](sindresorhus/got@v11.8.3...v11.8.5)

---
updated-dependencies:
- dependency-name: got
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
…ot-11.8.5

Bump got from 11.8.3 to 11.8.5
… to fix issues with incorrect length checking. (#593)

Co-authored-by: dschueller-cs <[email protected]>
* FAB-4815 - Validate registry name on activation

* Fixed problems with adding and removing registries

Co-authored-by: dschueller-cs <[email protected]>
* Oops.  Enforced the MAX length but not the MIN.

* Missing semicolon.  Sigh.

* A more concise implementation, and fixed problems with repeated validation failures.

Co-authored-by: dschueller-cs <[email protected]>
* initial commit

* consistent error message

* consistent list returns

* added test

* fixing integration tests

Co-authored-by: Akarsh <[email protected]>
* - Use dayjs over momentjs, moment is in maintenance mode needed better date parsing..
- Modify task list to support tasks from mongo
- Fix gh-got & got vuln

Add --schduled to tasks list cli to show scheduled tasks

Added command for schedule task, pause, resume

Clean task list with scheduled tasks

FAB-4477 sort options work for both other/new task apis

* Tweak pause/resume cli option to make wording less confusing

dayjs.extend(relativeTime);

const TASK_LIST_TABLE = [
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we keep the table format definition in utils.js. Moving TASK_LIST_TABLE and SCHED_LIST_TABLE there would be consistent

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree on the comment but I did not make the changes.. These got pulled in as there was a bad merge from develop from my end. Fixed the merge.

@yreddy-CS yreddy-CS requested a review from akprasad-cs July 14, 2022 04:52
@yreddy-CS yreddy-CS merged commit 2663a63 into develop Jul 14, 2022
@jgielstra-cs jgielstra-cs deleted the tasks-err-log branch July 15, 2022 18:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants