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

Droth 3541 separate queries for valid and expired links #2344

Conversation

SWJPTulijoki
Copy link
Contributor

Vaikuttaisi toimivan devillä ok.

@anttiahopeltositowise
Copy link
Contributor

Tämä taitaa vaatia testaamista, QA:lla oliko puhetta otetaanko tämä mukaan tämän päivän julkaisuun?

@SWJPTulijoki
Copy link
Contributor Author

Tämä taitaa vaatia testaamista, QA:lla oliko puhetta otetaanko tämä mukaan tämän päivän julkaisuun?

Dailyssa taidettiin puhua, että voidaan ottaa tai olla ottamatta. Migraatioiden näkökulmasta vois olla hyvä ottaa, niin ei jäis roikkumaan toi constraint.

@SWJPTulijoki SWJPTulijoki marked this pull request as draft December 5, 2022 07:24
@SWJPTulijoki
Copy link
Contributor Author

Ei oteta tätä vielä tulevaan julkaisuun, joten muutettu draftiksi.

Base automatically changed from NextRelease to master December 5, 2022 09:08
@sasuolanderSito sasuolanderSito changed the base branch from master to NextRelease December 5, 2022 09:09
@reetta-kivisto reetta-kivisto changed the base branch from NextRelease-old7-12 to NextRelease December 7, 2022 09:48
@SWJPTulijoki SWJPTulijoki marked this pull request as ready for review December 13, 2022 06:37
Copy link
Contributor

@sasuolanderSito sasuolanderSito left a comment

Choose a reason for hiding this comment

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

ok

@SWJPTulijoki SWJPTulijoki merged commit 82fa5a1 into NextRelease Dec 13, 2022
@SWJPTulijoki SWJPTulijoki deleted the DROTH-3541_separate_queries_for_valid_and_expired_links branch March 20, 2024 06:01
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.

3 participants