https://linen.dev logo
#contributing-to-airbyte
Title
# contributing-to-airbyte
j

Jared Rhizor (Airbyte)

11/09/2020, 6:06 PM
What's the point of
documentationUrl
in source/destination destinations?
u

user

11/09/2020, 6:06 PM
The spec command is already expected to return a real documentation URL.
u

user

11/09/2020, 6:06 PM
u

user

11/09/2020, 6:07 PM
which isn't used anywhere or particularly useful (as far as I can tell)?
u

user

11/09/2020, 6:07 PM
If it is used somewhere, shouldn't it use what the spec is returning instead?
u

user

11/09/2020, 6:16 PM
That’s intended to be the URL pointing to available image tags in the Admin UI
u

user

11/09/2020, 6:17 PM
and change log
u

user

11/09/2020, 6:18 PM
there's no repo overview there though
u

user

11/09/2020, 6:18 PM
and changelogs would presumably be in the root airbyte project?
u

user

11/09/2020, 6:19 PM
And can't tags be inferred from
airbyte/integration-singer-csv-destination
alone and the default registry?
u

user

11/09/2020, 6:20 PM
tags meaning all available tags
u

user

11/09/2020, 6:20 PM
For the consumer to find
u

user

11/09/2020, 6:21 PM
Why would a user go to that link to look for tags?
u

user

11/09/2020, 6:21 PM
we can determine what the tags are but it’s supposed to give an easy way for them to find out what tags exist. The missing piece is a change log. imo we should put changelogs in the root repo and then maybe change these urls to point there
u

user

11/09/2020, 6:22 PM
I thought the UI would automatically show available tags?
u

user

11/09/2020, 6:22 PM
It doesn’t do that at the moment
u

user

11/09/2020, 6:22 PM
ah
u

user

11/09/2020, 6:22 PM
so are we expecting to maintain up to date descriptions of each point version of integrations?
u

user

11/09/2020, 6:23 PM
if so that seems like it should exist in the same format as the source definition with some additional metadata for that version
u

user

11/09/2020, 6:23 PM
if we need a programattically accessible changelog
u

user

11/09/2020, 6:23 PM
so are we expecting to maintain up to date descriptions of each point version of integrations?
only in documentation at the moment
u

user

11/09/2020, 6:24 PM
basically a change log
u

user

11/09/2020, 6:27 PM
I'm still a bit confused how this helps a user select a tag they want
u

user

11/09/2020, 6:27 PM
wouldn't they always want latest unless they wanted to revert after upgrading?
u

user

11/09/2020, 6:27 PM
I can't imagine consciously choosing a specific older version otherwise
u

user

11/09/2020, 6:29 PM
assuming airbyte ships with the latest versions, this means people running airbyte instances would need to manually upgrade their connectors
u

user

11/09/2020, 6:29 PM
so you only upgrade when you specifically choose to
u

user

11/09/2020, 6:30 PM
is the concern that this is the intended behavior, or that the UX doesn’t help the user navigate it?
u

user

11/09/2020, 6:30 PM
I definitely think the latter is true right now
u

user

11/09/2020, 6:31 PM
I agree you should only upgrade when you choose to, but people would generally upgrade from current to latest if they want to upgrade
u

user

11/09/2020, 6:31 PM
the UX doesn't seem to help and the changelog won't help either if the pattern I'm describing is the primary use case
u

user

11/09/2020, 6:32 PM
upgrading to latest?
u

user

11/09/2020, 6:32 PM
yeah
u

user

11/09/2020, 6:32 PM
I don't think people will search through a changelog looking for the feature they wanted
u

user

11/09/2020, 6:32 PM
and upgrade to that
u

user

11/09/2020, 6:32 PM
vs going to the latest that also includes the feature they wanted
u

user

11/09/2020, 6:33 PM
right so in that case the user upgrades to the latest version
u

user

11/09/2020, 6:33 PM
unless you mean
latest
u

user

11/09/2020, 6:33 PM
seems like it'd mostly be bug fixes or changes to the underlying integration
u

user

11/09/2020, 6:33 PM
which is undesirable
u

user

11/09/2020, 6:33 PM
yeah I mean latest as in highest semantic version, not
latest
tag
u

user

11/09/2020, 6:34 PM
I'd prefer to have global airbyte changelogs instead of integration-specific ones
u

user

11/09/2020, 6:34 PM
wouldn’t having a changelog help them figure out the latest version if they don’t care to comb through every line in changelog?
u

user

11/09/2020, 6:34 PM
ah
u

user

11/09/2020, 6:34 PM
Just due to ease of management