Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Task] UI Package - Publish (without Git) #39289

Open
ashit-rath opened this issue Feb 14, 2025 · 0 comments
Open

[Task] UI Package - Publish (without Git) #39289

ashit-rath opened this issue Feb 14, 2025 · 0 comments
Assignees
@subrata71
Labels
Backend This marks the issue or pull request to reference server code Packages & Git Pod All issues belonging to Packages and Git Packages Pod issues that belong to the packages pod Task A simple Todo

Comments

@ashit-rath
Copy link
Contributor

No description provided.

@ashit-rath ashit-rath added the Packages & Git Pod All issues belonging to Packages and Git label Feb 14, 2025
@github-actions github-actions bot removed the Packages & Git Pod All issues belonging to Packages and Git label Feb 14, 2025
@ashit-rath ashit-rath assigned ashit-rath and subrata71 and unassigned ashit-rath Feb 14, 2025
@ashit-rath ashit-rath added Backend This marks the issue or pull request to reference server code Task A simple Todo Packages Pod issues that belong to the packages pod Packages & Git Pod All issues belonging to Packages and Git labels Feb 14, 2025
ALOK9442 pushed a commit to ALOK9442/appsmith that referenced this issue Feb 16, 2025
@ankitakinger
chore: Refactoring usage of editorType to ideType to remove code …
c223c50 
…duplication (appsmithorg#38778)

## Description

Refactoring usage of `editorType` to `ideType` to remove code
duplication

Fixes [appsmithorg#38286](appsmithorg#38286)
[appsmithorg#39289](appsmithorg#38289)

## Automation

/ok-to-test tags="@tag.All"

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/12878729200>
> Commit: dfe8f5c
> <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=12878729200&attempt=2"
target="_blank">Cypress dashboard</a>.
> Tags: `@tag.All`
> Spec:
> <hr>Tue, 21 Jan 2025 06:50:04 UTC
<!-- end of auto-generated comment: Cypress test results  -->


## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [ ] No


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

## Release Notes

- **New Features**
- Introduced a new IDE type context to replace the previous editor type
system
	- Enhanced type safety for IDE-related operations

- **Improvements**
	- Refactored editor type handling across multiple components
	- Updated utility functions to work with new IDE type
	- Simplified permission and context management

- **Code Quality**
	- Removed deprecated editor type hooks and constants
	- Standardized IDE type determination across the application
	- Improved import and module organization

- **Performance**
	- Streamlined type checking and context retrieval mechanisms

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
ALOK9442 pushed a commit to ALOK9442/appsmith that referenced this issue Feb 16, 2025
@ankitakinger
chore: Replace entities in entity explorer with new ADS templates (ap…
8f5aad9 
…psmithorg#38750)

## Description

Replace entities in entity explorer with new ADS templates

Fixes [appsmithorg#38286](appsmithorg#38286)
[appsmithorg#39289](appsmithorg#38289)

## Automation

/ok-to-test tags="@tag.All"

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/12945261277>
> Commit: bf508e2
> <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=12945261277&attempt=2"
target="_blank">Cypress dashboard</a>.
> Tags: `@tag.All`
> Spec:
> <hr>Fri, 24 Jan 2025 08:55:33 UTC
<!-- end of auto-generated comment: Cypress test results  -->


## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [ ] No


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

## Summary by CodeRabbit

## Release Notes

- **New Features**
  - Added new context menu components for JavaScript and Query entities.
- Introduced enhanced entity item rendering for JavaScript and Query
collections.
  - Implemented more granular permission-based context menu actions.
- Added new components: `Rename`, `Copy`, `Move`, `Delete`,
`ShowBindings`, `Prettify`, and `EntityContextMenu`.

- **Improvements**
  - Streamlined context menu functionality across editor interfaces.
  - Enhanced user permissions handling for entity actions.
  - Improved modularity of editor components.
- Updated rendering logic for JavaScript and Query lists based on
feature flags.

- **Bug Fixes**
  - Refined component prop management.
- Updated navigation and analytics event logging for entity
interactions.

- **Refactoring**
  - Simplified component structures.
  - Removed deprecated prop usage.
  - Consolidated import and export statements.

These changes primarily focus on improving the user experience and
developer flexibility within the application's editor interfaces.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@subrata71 subrata71

Labels
Backend This marks the issue or pull request to reference server code Packages & Git Pod All issues belonging to Packages and Git Packages Pod issues that belong to the packages pod Task A simple Todo
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@subrata71 @ashit-rath