Fix clicking on a parent issue label removes it - #3237
Closed
Bahaatbb wants to merge 1 commit into
Closed
Conversation
Member
vamsikrishnamathala
pushed a commit
that referenced
this pull request
Aug 7, 2025
* [SILO-249] feat: notion importer phase 1 with zip manager and worker traversal (#3153) * feat: added notion zip manager code * feat: added endpoint for bulk create pages * feat: added page service in sdk * fix: temporary runner * feat: added notion zip file node types * feat: added file extractor and file tree creator methods * feat: added zip file manager class for interacting with extractors * feat: added key manager types * feat: added notion data migrator class for processing nodes * feat: added method to set map in the store * feat: added initialization of s3 in worker base * feat: registered notion worker in the manager * feat: added env variables for s3 * fix: remove unused services * fix: removed unnecessary package * fix: moved options into types * fix: cleanup and jsdoc comments * fix: renamed types and enums for consistency * fix: replaced console.log with logger * fix: added ttl to caches * fix: moved zip manager to be reusable * feat: added distributed counter for keeping count of leaf nodes * fix: broken down the notion importer into multiple phases * fix arrow function for key Co-authored-by: Surya Prashanth <prashantsurya002@gmail.com> * fix: eslint issues * fix: unused variables and expression call * fix: resolved eslint warnings --------- Co-authored-by: Surya Prashanth <prashantsurya002@gmail.com> * fix: build fix * [SILO-254] feat: notion importer phase two implementation (#3169) * feat: created endpoint for bulk operation on pages * feat: added phase two implementation of notion migrator * feat: added expected and ignored files list * feat: added method for getting a specific map field * fix: duplicate import * feat: added page api endpoint * feat: restructured and exported parser from core * feat: added notion content parser extensions * feat: added content parser initialization * feat: added cleanup function in base migrator * feat: refactored phases to include content parsing and cleanups * chore: added content parser config * feat: added cleanup function in zip manager to remove toc * feat: added bluebird as dependency * chore: remove test file * fix: formatted code * fix: decoupled content parser from `etl/core` to avoid circular dependency (#3204) * fix: decoupled parser from etl/core and moved to etl/parser * fix: modified linear importer for using parser from etl/parser * fix: updated notion importer content parser imports * [SILO-265] feat: Notion Importer UI and controller for generating presigned URL (#3237) * feat: added notion controller for handling credentials save and uploads * feat: added new keys and cached job and credential data * feat: added notion controller and s3 init script * feat: added UI changes for notion importer * feat: added translation file and feature flags * feat: added model changes to job and modified endpoint for pages * feat: added start time to job * fix: silo build failing * feat: added notion importer UI component for uploading * feat: added new extension for processing bookmark links * fix: coderabbit suggestions * fix: build fixes * fix: condition * fix: added page bulk endpoint under hmac and apiclient * fix: root page when there is only one directory * fix: added root directory path and error handling * fix: add html page for each directory if not present * feat: added feature flags for notion importer * feat: added translations for all languagaes * feat: added translations * fixed: translations for russian * [SILO-302] feat: migrated notion zip upload endpoints to apiserver (#3327) * feat: added endpoint for silo uploads for notion importer * feat: refactored controller to add start import route * fix: removed policy for ttl in s3 * feat: modified frontend for using new endpoints from apiserver for uploads * chore: added permission to silo asset endpoint * feat: added notion importer feature flag to asset endpoint * fix: used existing assets v2 for updating the asset upload * fix: removed unused endpoints from silo asset view * fix: moved notion importer into new settings page --------- Co-authored-by: Surya Prashanth <prashantsurya002@gmail.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
This change resolves issue #3229 by moving the closing behavior inside the x instead of the label