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

🪟 🔧 Move 'Example values' into intl #19446

Merged
merged 1 commit into from
Nov 15, 2022

Conversation

lmossman
Copy link
Contributor

@lmossman lmossman commented Nov 15, 2022

What

The "Example values" header in the label info tooltip was originally not implemented in an intl-friendly way. This PR corrects that.

There should be no user-visible changes from this PR.

@octavia-squidington-iv octavia-squidington-iv added area/platform issues related to the platform area/frontend Related to the Airbyte webapp labels Nov 15, 2022
@lmossman lmossman marked this pull request as ready for review November 15, 2022 19:26
@lmossman lmossman requested a review from a team as a code owner November 15, 2022 19:26
Copy link
Contributor

@timroes timroes left a comment

Choose a reason for hiding this comment

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

Code LGTM. Have not checked out locally.

@lmossman lmossman merged commit b25b631 into master Nov 15, 2022
@lmossman lmossman deleted the lmossman/fix-example-values-intl branch November 15, 2022 20:22
colesnodgrass pushed a commit that referenced this pull request Nov 16, 2022
colesnodgrass added a commit that referenced this pull request Nov 30, 2022
* wip; add micronaut

* add additional json deserializer methods

* wip; converting to micronaut

* misc cleanup

* wip; broken

* wip; still broken

* wip

* formatting

* minor code cleanup; no actual changes

* wip; still broken

* removed commented out code; no longer broken

* wip; clean-up micronaut code

* cleanup; format

* fix pmd issues

* remove unused file

* init ApplicationTest

* edited link (#19444)

* move 'Example values' into intl (#19446)

* Revert "Update action.yml (#19416)" (#19450)

This reverts commit 78fb528.

* Notifications Workflow (#18735)

* notification workflow

* Bmoric/remove unused code (#19188)

* Tmp

* Move when the deletion is performed

* Re-enable disable test

* PR comments

* Use cancel

* rename

* Fix test and version check position

* remove unused temporal deletion code

* Remove false todo

* Rm repeated test

* Rm unused import

* Make sure that long running activity are not retried (#19452)

* Parse list of dicts in json_schema_helper.find_nodes() (#19386)

* Get test on nested list/dict passing - use index to query next object for list

* Fix flakecheck

* Test that get_node provides correct value

* Improve test and test cases

* Rewrite method for better comprehension

* Add test for base-level key. Rewrite method for comprehension and handling this case

* adding tests

* fix test

* formatting

* remove unused dependencies

* add missing test resource

* format

* add missing test resource (real)

* format

* add back protocol-models dep

* format

* pr feedback; log stacktrace

Co-authored-by: Sophia Wiley <106352739+sophia-wiley@users.noreply.github.com>
Co-authored-by: Lake Mossman <lake@airbyte.io>
Co-authored-by: Topher Lubaway <asimplechris@gmail.com>
Co-authored-by: Anne <102554163+alovew@users.noreply.github.com>
Co-authored-by: Benoit Moriceau <benoit@airbyte.io>
Co-authored-by: Ella Rohm-Ensing <erohmensing@gmail.com>
akashkulk pushed a commit that referenced this pull request Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to the Airbyte webapp area/platform issues related to the platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants