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.

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

Partition Table Editor UI (VSC-439) #170

Merged
merged 83 commits into from
Dec 30, 2020
Merged

Partition Table Editor UI (VSC-439) #170

merged 83 commits into from
Dec 30, 2020

Conversation

pwmb
Copy link
Contributor

@pwmb pwmb commented Sep 16, 2020

Add UI for editing and flashing Partition Table

Tasks

  • Add support for csv & bins files
  • Add Save to write to fs (csv)
  • Load content from csv on startup
  • Update Docs steps and Troubleshoot
  • Remove the bin r/w
  • Add input validation
  • Add i18n
  • Add build & flash button
  • Once approve add final screen shot in docs

Screenshots

Screen Shot 2020-10-21 at 12 19 55 PM

pwmb and others added 30 commits March 16, 2020 18:19
Merge master from espressif/vscode-esp-idf-extension
* initial esp-adf support

* fix images in examples html src

* choose framework before loading examples

* fix some images in markdown html

* fix svg images

* mv types marked to dev deps

* add esp-mdf and examples

* set MDF to MDF_PATH env variable
Bumps [bl](https://github.com/rvagg/bl) from 4.0.2 to 4.0.3.
- [Release notes](https://github.com/rvagg/bl/releases)
- [Commits](rvagg/bl@v4.0.2...v4.0.3)

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

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@github-actions
Copy link

Pull request has been marked as stale since there are no activities, and this will be closed in 5 days if there are no further activities

@github-actions github-actions bot added the stale Stale PR or Issue label Nov 15, 2020
@pwmb pwmb removed the stale Stale PR or Issue label Nov 15, 2020
@github-actions
Copy link

github-actions bot commented Dec 4, 2020

Pull request has been marked as stale since there are no activities, and this will be closed in 5 days if there are no further activities

@github-actions github-actions bot added the stale Stale PR or Issue label Dec 4, 2020
@pwmb pwmb removed the stale Stale PR or Issue label Dec 4, 2020
@pwmb
Copy link
Contributor Author

pwmb commented Dec 18, 2020

@brianignacio5 are you implementing the NVS Editor based on this PR? If so we can review and merge it first.

@brianignacio5
Copy link
Collaborator

Sure thing, I originally wanted to reuse the most of the code (just picking some parts of the code), maybe in future we can refactor to have common components between different webviews. Will review this PR today.

Copy link
Collaborator

@brianignacio5 brianignacio5 left a comment

Choose a reason for hiding this comment

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

Left some suggestions about CSV files and load.

src/views/partition-table/util.ts Outdated Show resolved Hide resolved
src/views/partition-table/store/index.ts Show resolved Hide resolved
src/espIdf/partition-table/index.ts Show resolved Hide resolved
Copy link
Collaborator

@brianignacio5 brianignacio5 left a comment

Choose a reason for hiding this comment

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

LGTM thanks for the changes.

@pwmb pwmb merged commit 98cc4d2 into espressif:master Dec 30, 2020
@pwmb pwmb deleted the feature/partition-table-editor branch December 30, 2020 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature A new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants