Skip to content

Update bug_report.md#2014

Closed
jcqli wants to merge 1 commit intomasterfrom
jcqli-patch-1
Closed

Update bug_report.md#2014
jcqli wants to merge 1 commit intomasterfrom
jcqli-patch-1

Conversation

@jcqli
Copy link

@jcqli jcqli commented Sep 6, 2024

Add app versions to bug template. Towards #2040

@gino-m PTAL

A clear and concise description of what the bug is.

**App versions**
Android:
Copy link
Collaborator

Choose a reason for hiding this comment

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

As this for is mainly used for bugs in the web UI, perhaps we don't need Android in the template?

Also, I just realized we don't actually have build numbers / versions in the web UI, only in Android. Perhaps we should add an FR to show the version in the about screen before adding this to the template?

Copy link
Author

Choose a reason for hiding this comment

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

Filed #2015

I think it would be useful to include both when applicable, since a mismatch of versions could also be a root cause of bugs, or there could be a cross-app interaction that needs debugging. WDYT?

@jcqli
Copy link
Author

jcqli commented Sep 6, 2024

@gino-m should we close this PR or leave it in draft until #2015 is fixed?

@gino-m
Copy link
Collaborator

gino-m commented Sep 6, 2024

@gino-m should we close this PR or leave it in draft until #2015 is fixed?

We can leave it open, maybe add as a subtask of #2015.

@gino-m
Copy link
Collaborator

gino-m commented Dec 10, 2024

Closing this since we don't yet have versioned releases of the web interface.

@gino-m gino-m closed this Dec 10, 2024
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.

2 participants