Skip to content

Standardise name of tracker_file_extension#57

Merged
tpoliaw merged 1 commit into
mainfrom
extension_name
Jan 20, 2025
Merged

Standardise name of tracker_file_extension#57
tpoliaw merged 1 commit into
mainfrom
extension_name

Conversation

@tpoliaw

@tpoliaw tpoliaw commented Jan 17, 2025

Copy link
Copy Markdown
Collaborator

Instead of using fallback_extension and extension in different places. This is the file used by the GDA numtracker and should not be confused with any extension used for the scan files generated by the service.

@tpoliaw

tpoliaw commented Jan 17, 2025

Copy link
Copy Markdown
Collaborator Author

This should be merged before #54 which should then be updated to also use tracker_file_extension.

DiamondJoseph
DiamondJoseph previously approved these changes Jan 17, 2025

@DiamondJoseph DiamondJoseph left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

from what I understand and to sign off on naming

Instead of using fallback_extension and extension in different places.
This is the file used by the GDA numtracker and should not be confused
with any extension used for the scan files generated by the service.
@tpoliaw tpoliaw merged commit 5197837 into main Jan 20, 2025
@tpoliaw tpoliaw deleted the extension_name branch January 20, 2025 12:12
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