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.

Sign up for GitHub

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

Jump to bottom

454 restructuring heerlijk helder #497

Open
wants to merge 150 commits into
base: uat
Choose a base branch
from
Open

454 restructuring heerlijk helder #497

wants to merge 150 commits into from

Conversation

mvarewyck
Copy link
Collaborator

Restructuring of the public app.

We already checked build & run docker image for public & private app.

lcougnaud and others added 30 commits October 14, 2024 11:16
support local data (reportingGrofwild-data-path env var)
1f6c2ed
add carousel (WIP)
a3acb77
setupS3: add AWS session token
1a55000
update S3 bucket
3f241ae
update front page (contact, version) + improve style
02def39
add specie page (WIP)
dbc912d
specie page: finalize layout + use module + redirect to Home page
2594025
fix initialization of 'specie' page
9886084
specie page: use radio buttons instead of shinybs::cards for
94139b7 
interactivity
fix redirect home logic
f42a4ef
fix alignment specie cards
b43afa4
add skeleton afshot category page
f690491
fix typos
d227352
category 'afshot' page: improve reactivity
a6b1027
clean-up specie page redirection
80b15f2
category page: add logic plot sub-page
015615e
test debug namespace issue plot module
c11489e
@mvarewyck
fix NS issues submodules plots + fullpage slickr + fix
187c393 
bslib::card_image()
@mvarewyck
connection to S3 using tokens (similar to alienspecies)
3efd467
afshot page: add barplot per province
22dd3fd
Merge branch '454_restructuring-heerlijk-helder' of https://scm.opena…
840d307 
…nalytics.eu/git/reporting-rshiny-grofwildjacht.git into 454_restructuring-heerlijk-helder
category afschot: import title/description cards from uiText + fix
22b1558 
countYearProvince plot
category page: update header upon tab selection + finetuning
60ccad9
category: add reset tab based on path
67c06da
streamline creation of plot title + description
e118968
category afshot: add yearlyShotAnimals plot
edd9d28
category: add map afshot per regio + fix update path subcategory
439fcb5
category: add table afshot per leeftijdcategorie
63fc68f
category: add plot count by age category
0020fd0
category: add afschot plot per jachtmethode
f905c37
mvarewyck and others added 27 commits January 15, 2025 17:42
@mvarewyck
fix hash diersoort + add link to email
11ef761
@mvarewyck
add MWE for css styling
ca9e267
@mvarewyck
css styling for background-color and white space
9d7360b
@lcougnaud
Specie -> Diersoort for tab update
1653af6
@lcougnaud
fix update hash to specie only
ed5a3f2
@lcougnaud
fix update hash from subcategory, then specie -> category
2f88001
@lcougnaud
fix update hash from plot, then specie -> category -> subcategory
e95d043
@lcougnaud
generalize code: subcategory tile button -> output page
0585ac1
@mvarewyck
fix trendYearFlanders plot for schade
5e2ec51
@mvarewyck
remove double vertical scrollbar
c24397f
@mvarewyck
Merge branch '454_restructuring-heerlijk-helder' of https://scm.opena…
e8dd4c9 
…nalytics.eu/git/reporting-rshiny-grofwildjacht.git into 454_restructuring-heerlijk-helder
@lcougnaud
print progress messages only upon doDebug = TRUE
3c84f98
@mvarewyck
fix navigation: selecting same page 2nd time
1b34af5
@mvarewyck
Merge branch '454_restructuring-heerlijk-helder' of
6dc18d5 
https://scm.openanalytics.eu/git/reporting-rshiny-grofwildjacht.git into
454_restructuring-heerlijk-helder
@mvarewyck
update dockerfile + deployment instructions
6c4d5f5
@mvarewyck
increase margin description cards
414c56f
@mvarewyck
fix alignment of radio-tiles-images
08c5723
@lcougnaud
add extra space bottom/top of the button in category/subcategory tiles
cd51988
@lcougnaud
add extra space bottom/top of the button in category/subcategory tiles +
2dc5c1f 
extra space top image
@mvarewyck
fix WBE page: css + descriptions + radio-tiles + move wellPanels
4f5b18e
@mvarewyck
Merge branch '454_restructuring-heerlijk-helder' of https://scm.opena…
60a665c 
…nalytics.eu/git/reporting-rshiny-grofwildjacht.git into 454_restructuring-heerlijk-helder
@lcougnaud
extract available outputs from the data + blacklist (#3)
0562693
@lcougnaud
fix uiText quote
28404e8
@lcougnaud
Merge branch '454_restructuring-heerlijk-helder' of https://scm.opena…
605bbbb 
…nalytics.eu/git/reporting-rshiny-grofwildjacht.git into 454_restructuring-heerlijk-helder
@lcougnaud
remove blank category tiles for specie with subset of categories
efc3776
@lcougnaud
fixes Rd for R CMD check
abf3b5a
@mvarewyck
version dump
03f5d57
@mvarewyck mvarewyck requested a review from soriadelva February 14, 2025 16:24
@mvarewyck mvarewyck self-assigned this Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@soriadelva soriadelva Awaiting requested review from soriadelva

At least 1 approving review is required to merge this pull request.

Assignees

@mvarewyck mvarewyck

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mvarewyck @SanderDevisscher @lcougnaud