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

Feather ESP32-S2 Reverse TFT: Update PID to match internal list #7478

Merged
merged 2 commits into from Jan 26, 2023

Conversation

jepler
Copy link
Member

@jepler jepler commented Jan 23, 2023

@ladyada please check this and leave a review so it's right this time :)

@jepler jepler requested a review from ladyada January 23, 2023 19:15
@jepler jepler changed the title Update PID to match internal list Feather ESP32-S2 Reverse TFT Update PID to match internal list Jan 23, 2023
@jepler jepler changed the title Feather ESP32-S2 Reverse TFT Update PID to match internal list Feather ESP32-S2 Reverse TFT: Update PID to match internal list Jan 23, 2023
@ladyada
Copy link
Member

ladyada commented Jan 23, 2023

yes!

@dhalbert
Copy link
Collaborator

@jepler I'm confused, now there is a dupe between:
adafruit_feather_esp32s2_reverse_tft
and
adafruit_feather_esp32s2_tftback_nopsram

@ladyada
Copy link
Member

ladyada commented Jan 25, 2023

adafruit_feather_esp32s2_tftback_nopsram was never/will never be made

@jepler
Copy link
Member Author

jepler commented Jan 25, 2023

@dhalbert should we remove the other board definition (it's been there for quite awhile!) or permit the duplicate ID?

@dhalbert
Copy link
Collaborator

Let's remove adafruit_feather_esp32s2_tftback_nopsram, and any sign of it on circuitpython.org.

@dhalbert dhalbert merged commit 5fb0333 into adafruit:main Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants